This is an archive of the discontinued LLVM Phabricator instance.

[LegalizeDAG] Expand SADDO/SSUBO using SADDSAT/SSUBSAT (PR37763)
ClosedPublic

Authored by RKSimon on Feb 25 2019, 10:36 AM.

Details

Summary

If SADDSAT/SSUBSAT are legal, then we can expand SADDO/SSUBO by performing a ADD/SUB and a SADDO/SSUBO and then compare the results.

I looked at doing this for UADDO/USUBO as well but as we don't have to do as many range comparisons I didn't see any/much benefit.

Diff Detail

Repository
rL LLVM

Event Timeline

RKSimon created this revision.Feb 25 2019, 10:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 25 2019, 10:36 AM
Herald added a subscriber: jdoerfert. · View Herald Transcript
nikic accepted this revision.Feb 25 2019, 12:36 PM

LGTM

This revision is now accepted and ready to land.Feb 25 2019, 12:36 PM
This revision was automatically updated to reflect the committed changes.