This is an archive of the discontinued LLVM Phabricator instance.

Pre-commit test cases for DAG combine folds of SSHLSAT/USHLSAT -> SHL
ClosedPublic

Authored by bjope on Feb 2 2022, 2:32 AM.

Details

Summary

It should be possible to replace SSHLSAT and USHLSAT with SHL when
it is known that no saturation will take place (e.g. by analysing
known sign bits in the first shift operand).

Diff Detail

Event Timeline

bjope requested review of this revision.Feb 2 2022, 2:32 AM
bjope created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 2 2022, 2:32 AM
dmgreen accepted this revision.Feb 6 2022, 7:59 AM
dmgreen added a subscriber: dmgreen.

New test are always fine - feel free to commit without review as needed, you seem to know what you are doing.

llvm/test/CodeGen/AArch64/sshl_sat.ll
131

These are not using %y :)

This revision is now accepted and ready to land.Feb 6 2022, 7:59 AM
This revision was landed with ongoing or failed builds.Feb 6 2022, 10:00 AM
This revision was automatically updated to reflect the committed changes.