Page MenuHomePhabricator

[DAG] Convert truncstore(extend(x)) back to store(x)
ClosedPublic

Authored by dmgreen on Jan 21 2022, 9:16 AM.

Details

Summary

Pulled out of D106237, this folds truncstore(extend(x)) back to store(x) if the original store was legal. This can come up due to the order we fold nodes. A fold from X86 needs to be adjusted to prevent infinite loops, to have it pick the operand of a trunc more directly.

Diff Detail

Event Timeline

dmgreen created this revision.Jan 21 2022, 9:16 AM
dmgreen requested review of this revision.Jan 21 2022, 9:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 21 2022, 9:16 AM
Herald added a subscriber: wdng. · View Herald Transcript

X86 change LGTM.

RKSimon accepted this revision.Jan 21 2022, 10:15 AM

LGTM - cheers

This revision is now accepted and ready to land.Jan 21 2022, 10:15 AM
This revision was automatically updated to reflect the committed changes.