This is an archive of the discontinued LLVM Phabricator instance.

Let findCommutedOpIndices take const MachineInstr&
ClosedPublic

Authored by hvdijk on Aug 13 2019, 5:12 AM.

Details

Summary

Neither the base implementation of findCommutedOpIndices nor any in-tree
target modifies the instruction passed in and there is no reason why
they would in the future.

Diff Detail

Event Timeline

hvdijk created this revision.Aug 13 2019, 5:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 13 2019, 5:12 AM
RKSimon accepted this revision.Sep 25 2019, 3:22 AM

LGTM - thanks

This revision is now accepted and ready to land.Sep 25 2019, 3:22 AM

Thanks! I do not have commit access, can somebody commit it for me?

This revision was automatically updated to reflect the committed changes.