craig.topper added a comment.
There's a separate review for X86.d https://reviews.llvm.org/D34828
https://reviews.llvm.org/D34842
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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
setFeatureEnabledIm
craig.topper created this revision.
This adds to RDNRND to Goldmont as I believe it has that feature in addition to
RDSEED.
I don't know how to test the MOVBE part since we don't have a __MOVBE__ macro
nor do we need one. The backend will already infer the feature from the CPU
name. We do have
