This is an archive of the discontinued LLVM Phabricator instance.

Simplify CpuIs code to use include from LLVM
ClosedPublic

Authored by erichkeane on Nov 14 2017, 3:35 PM.

Details

Summary

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.

Diff Detail

Event Timeline

erichkeane created this revision.Nov 14 2017, 3:35 PM
erichkeane added inline comments.
lib/CodeGen/CGBuiltin.cpp
4630

Note that these two are required, since there is an llvm::ARM and a clang::ARM, and the top of this does a 'using namespace clang' and a 'using namespace llvm'.

Re-ran format on a few lines to make them more sane.

This revision is now accepted and ready to land.Nov 14 2017, 3:59 PM
This revision was automatically updated to reflect the committed changes.