[PATCH] D21268: [x86] translate SSE packed FP comparison builtins to IR
Sanjay Patel via cfe-commits
cfe-commits at lists.llvm.org
Mon Jun 13 09:44:44 PDT 2016
spatel added a comment.
In http://reviews.llvm.org/D21268#455679, @RKSimon wrote:
> Eeep that's certainly a lot more work than just adding a few extra cases! Please add a TODO explaining what we need to do?
I don't know what the answer is yet...looks like this is going to require (a lot of) testing to sort out.
> If there is a problem with the header documentation please can you raise a bugzilla and CC Katya Romanova.
Filed PR28110:
https://llvm.org/bugs/show_bug.cgi?id=28110
The initial test says that AMD's documentation is wrong: cmpps with immediate '8' produces a different answer than immediate '0' running on Jaguar.
http://reviews.llvm.org/D21268
More information about the cfe-commits
mailing list