[PATCH] D19658: [X86] Add -m[no-]x87 and -m[no-]80387 options to control FeatureX87

Eric Christopher via cfe-commits cfe-commits at lists.llvm.org
Sat Apr 30 16:11:45 PDT 2016


echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.

Nice. If you wouldn't mind splitting out into 3 patches (all ok):

a) The initial testcase,
b) the mno-cx16 patch and associated test
c) The x87 part and associated test.

You'll also want to put in that there's definitely a bit of "last one wins" on the testcases and so that the test case isn't exhaustive.

Thanks!

-eric


http://reviews.llvm.org/D19658





More information about the cfe-commits mailing list