sdardis accepted this revision. sdardis added a comment. This revision is now accepted and ready to land.
The new define also requires a test in test/Preprocessor/init.c - test that by default the new define isn't present, and in some case, when supplied with the -mno-madd4 that it is present. LGTM with that change. https://reviews.llvm.org/D33401 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits