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
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
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
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:
---
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