| [eigen-commits] commit/eigen: markdryan: Remove reinterpret_cast from AVX512 complex implementation |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/eigen-commits Archives
]
- To: eigen-commits@xxxxxxxxxxxxxxxxxxx
- Subject: [eigen-commits] commit/eigen: markdryan: Remove reinterpret_cast from AVX512 complex implementation
- From: Bitbucket <commits-noreply@xxxxxxxxxxxxx>
- Date: Fri, 11 Jan 2019 16:06:14 +0000 (UTC)
- Dkim-signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=bitbucket.org; h=mime-version:content-type:content-transfer-encoding:subject:from:to; s=s1; bh=DUwMnOhk+c1garvx+P+w7UMyfow=; b=MbXCK80m9bnLRQzfUKaeX6V PKv/+2xmwqG1HgRZaTNW76qE+3QNxVpNVlgzXHNjedhw67daX/fuc5X6RrRbN9YI fOIfkMqBd03mlThXMC2Jz+XSqkWudsA62nOYr4FNX6xhPZ7rDyCcsZDHNKNlFvGf uq+ASzGizxI5FW7D4LME=
1 new commit in eigen:
https://bitbucket.org/eigen/eigen/commits/9a28171b87b2/
Changeset: 9a28171b87b2
User: markdryan
Date: 2019-01-11 13:02:09+00:00
Summary: Remove reinterpret_cast from AVX512 complex implementation
The reinterpret_casts used in ptranspose(PacketBlock<Packet8cf,4>&)
ptranspose(PacketBlock<Packet8cf,8>&) don't appear to be working
correctly. They're used to convert the kernel parameters to
PacketBlock<Packet8d,T>& so that the complex number versions of
ptranspose can be written using the existing double implementations.
Unfortunately, they don't seem to work and are responsible for 9 unit
test failures in the AVX512 build of tensorflow master. This commit
fixes the issue by manually initialising PacketBlock<Packet8d,T>
variables with the contents of the kernel parameter before calling
the double version of ptranspose, and then copying the resulting
values back into the kernel parameter before returning.
Affected #: 1 file
Repository URL: https://bitbucket.org/eigen/eigen/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.