Page MenuHomePhabricator

[ARM] Removed hasSideEffects from signed/unsigned saturates
ClosedPublic

Authored by MeeraN on Sep 30 2020, 2:13 AM.

Details

Summary

Removed hasSideEffects from SSAT and USAT so that they are no longer
marked as unpredictable

Diff Detail

Event Timeline

MeeraN created this revision.Sep 30 2020, 2:13 AM
MeeraN requested review of this revision.Sep 30 2020, 2:13 AM
MeeraN removed a project: Restricted Project.
Herald added a project: Restricted Project. · View Herald Transcript

Sorry, didn't realise tags and reviewers were automatically added. Thought it was a mistake. Apologies for the few extra emails notifications that would have been sent.

dmgreen accepted this revision.Oct 1 2020, 12:20 AM

Yeah, Looks good to me. It brings us into line with the other instructions like qadd.

This revision is now accepted and ready to land.Oct 1 2020, 12:20 AM
This revision was landed with ongoing or failed builds.Oct 1 2020, 7:56 AM
This revision was automatically updated to reflect the committed changes.