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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to