[PATCH] D40054: Simplify CpuIs code to use include from LLVM

2017-11-14 Thread Erich Keane via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL318234: Simplify CpuIs code to use include from LLVM (authored by erichkeane). Changed prior to commit: https://reviews.llvm.org/D40054?vs=122936&id=122942#toc Repository: rL LLVM https://reviews.ll

[PATCH] D40054: Simplify CpuIs code to use include from LLVM

2017-11-14 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/D40054 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/li

[PATCH] D40054: Simplify CpuIs code to use include from LLVM

2017-11-14 Thread Erich Keane via Phabricator via cfe-commits
erichkeane updated this revision to Diff 122936. erichkeane added a comment. Re-ran format on a few lines to make them more sane. https://reviews.llvm.org/D40054 Files: lib/Basic/Targets/X86.cpp lib/CodeGen/CGBuiltin.cpp Index: lib/CodeGen/CGBuiltin.cpp

[PATCH] D40054: Simplify CpuIs code to use include from LLVM

2017-11-14 Thread Erich Keane via Phabricator via cfe-commits
erichkeane added inline comments. Comment at: lib/CodeGen/CGBuiltin.cpp:4630 case NEON::BI__builtin_neon_vsha1mq_u32: - case ARM::BI_MoveToCoprocessor: - case ARM::BI_MoveToCoprocessor2: + case clang::ARM::BI_MoveToCoprocessor: + case clang::ARM::BI_MoveToCoprocessor2: ---

[PATCH] D40054: Simplify CpuIs code to use include from LLVM

2017-11-14 Thread Erich Keane via Phabricator via cfe-commits
erichkeane created this revision. LLVM exposes a file in the backend (X86TargetParser.def) that contains information about the correct list of CpuIs values. This patch removes 2 of the copied and pasted versions of this list from clang and instead includes the data from the .def file. https://r