craig.topper updated this revision to Diff 118973.
craig.topper added a comment.
Only define "corei7" on nehalem/westmere to match gcc. Don't define anything
for the CPUs newer than that. Add comments to the CPUs where gcc has two sets
of defines and we have only one.
https://reviews.llvm.org/
chandlerc added inline comments.
Comment at: lib/Basic/Targets/X86.cpp:844-845
-// FIXME: Historically, we defined this legacy name, it would be nice to
-// remove it at some point. We've never exposed fine-grained names for
-// recent primary x86 CPUs, and we should
craig.topper added inline comments.
Comment at: lib/Basic/Targets/X86.cpp:844-845
-// FIXME: Historically, we defined this legacy name, it would be nice to
-// remove it at some point. We've never exposed fine-grained names for
-// recent primary x86 CPUs, and we shou
chandlerc added inline comments.
Comment at: lib/Basic/Targets/X86.cpp:844-845
-// FIXME: Historically, we defined this legacy name, it would be nice to
-// remove it at some point. We've never exposed fine-grained names for
-// recent primary x86 CPUs, and we should
craig.topper created this revision.
Herald added a subscriber: krytarowski.
We were using corei7 for a large swatch of Intel CPUs. gcc has a different
defines that more closely match the march flags. This updates to match. It
also fixes skylake-avx512 and adds silvermont in addition to slm.
h
