This is an archive of the discontinued LLVM Phabricator instance.

[X86] Move even more of our CPU to feature mapping switch to use fallthroughs
ClosedPublic

Authored by craig.topper on Sep 15 2017, 3:56 PM.

Details

Summary

This arranges more of the Intel and AMD CPUs into fallthrough positions based on their features. We may be able to merge this new AMD set with the BTVER or BDVER sets but I didn't look that closely.

Diff Detail

Event Timeline

RKSimon accepted this revision.Sep 16 2017, 2:32 AM

LGTM, there are a couple of other fall-throughs that could still be done.

lib/Basic/Targets/X86.cpp
141

CK_Pentium3/CK_C3_2/CK_PentiumM/CK_Pentium4/CK_x86_64 can be put below Yonah et al. (unless you're worried about including non-Intel enums)?

This revision is now accepted and ready to land.Sep 16 2017, 2:32 AM
This revision was automatically updated to reflect the committed changes.