[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