This is an archive of the discontinued LLVM Phabricator instance.

[X86] Add FeatureCMOV to athlon and athlon-tbird cpus.
ClosedPublic

Authored by craig.topper on Aug 25 2018, 7:02 PM.

Details

Summary

This matches gcc and one cpuid dump I found online. Given that these are considered 7th generation x86 CPU it seems likely they support cmov since cmov was added by Intel in their 6th generation.

Diff Detail

Repository
rL LLVM

Event Timeline

craig.topper created this revision.Aug 25 2018, 7:02 PM
RKSimon accepted this revision.Aug 26 2018, 5:45 AM

LGTM - thanks

test/CodeGen/X86/select.ll
4 ↗(On Diff #162571)

You might be able reduce the checks with more use of CHECK, CMOV and NO-CMOV common prefixes?

This revision is now accepted and ready to land.Aug 26 2018, 5:45 AM
This revision was automatically updated to reflect the committed changes.