This is an archive of the discontinued LLVM Phabricator instance.

[LegalizeTypes][ARM][X86] Change ExpandIntRes_ABS to use sra+xor+sub.
ClosedPublic

Authored by craig.topper on Mar 4 2022, 6:58 PM.

Details

Summary

Previously we used sra+add+xor if ADDCARRY is supported. This changes
to sra+xor+sub is SUBCARRY is available.

This is consistent with the recent change to the default expansion
in LegalizeDAG.

Diff Detail

Event Timeline

craig.topper created this revision.Mar 4 2022, 6:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 4 2022, 6:58 PM
craig.topper requested review of this revision.Mar 4 2022, 6:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 4 2022, 6:58 PM
RKSimon accepted this revision.Mar 7 2022, 7:19 AM

LGTM

This revision is now accepted and ready to land.Mar 7 2022, 7:19 AM
This revision was landed with ongoing or failed builds.Mar 7 2022, 11:28 AM
This revision was automatically updated to reflect the committed changes.