RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.
LGTM with a couple of comments.
On the LLVM side, X86.td GoldMont class looks like it needs RDRAND as well.
================
Comment at: lib/Basic/Targets.cpp:3310
setFeatureEnabledImpl(Features, "cx16", true);
- break;
+ break;
case CK_KNL:
----------------
Is it worth doing the reordering first as an NFC? Then add MOVBE?
https://reviews.llvm.org/D34842
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
