[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