RKSimon accepted this revision. RKSimon added a comment. This revision is now accepted and ready to land.
LGTM, there are a couple of other fall-throughs that could still be done. ================ Comment at: lib/Basic/Targets/X86.cpp:141 setFeatureEnabledImpl(Features, "sse2", true); setFeatureEnabledImpl(Features, "fxsr", true); break; ---------------- CK_Pentium3/CK_C3_2/CK_PentiumM/CK_Pentium4/CK_x86_64 can be put below Yonah et al. (unless you're worried about including non-Intel enums)? https://reviews.llvm.org/D37941 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits