[PATCH] D34842: [X86] Add RDRND to Goldmont. Add MOVBE to all Atom CPUs

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 30 09:30:13 PDT 2017


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





More information about the cfe-commits mailing list