This is an archive of the discontinued LLVM Phabricator instance.

[TypePromotion] Allow SExtInst as sink
Needs ReviewPublic

Authored by samparker on Feb 3 2022, 7:12 AM.



If these instructions are in the tree, then they're almost certainly going to need a sign extend anyway, so having a trunc in the IR to handle them shouldn't cause any harm. I also noticed some significantly better codegen when targeting arm on one of the IR tests.

Diff Detail

Event Timeline

samparker created this revision.Feb 3 2022, 7:12 AM
samparker requested review of this revision.Feb 3 2022, 7:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 3 2022, 7:12 AM