[PATCH] D39378: Remove x86,x86_32/64 from isValidFeatureName

2017-10-27 Thread Erich Keane via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL316781: Remove x86,x86_32/64 from isValidFeatureName (authored by erichkeane). Changed prior to commit: https://reviews.llvm.org/D39378?vs=120661&id=120663#toc Repository: rL LLVM https://reviews.ll

[PATCH] D39378: Remove x86,x86_32/64 from isValidFeatureName

2017-10-27 Thread Craig Topper via Phabricator via cfe-commits
craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D39378 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/li

[PATCH] D39378: Remove x86,x86_32/64 from isValidFeatureName

2017-10-27 Thread Erich Keane via Phabricator via cfe-commits
erichkeane created this revision. These are not valid values for this, and are pretty non-sensical, since LLVM doesn't understand them. https://reviews.llvm.org/D39378 Files: lib/Basic/Targets/X86.cpp Index: lib/Basic/Targets/X86.cpp =