This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Move add64/sub64 to VALU
ClosedPublic

Authored by rampitec on Aug 29 2023, 2:50 AM.

Details

Summary

This is NFCI as far as I can tell, but I see no reason not to do it.

Diff Detail

Event Timeline

rampitec created this revision.Aug 29 2023, 2:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 29 2023, 2:50 AM
rampitec requested review of this revision.Aug 29 2023, 2:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 29 2023, 2:50 AM
Herald added a subscriber: wdng. · View Herald Transcript
arsenm accepted this revision.Aug 29 2023, 4:40 AM

Is this even reachable?

This revision is now accepted and ready to land.Aug 29 2023, 4:40 AM

Is this even reachable?

I did not succeed to write a test reaching this.

This revision was landed with ongoing or failed builds.Aug 29 2023, 9:31 AM
This revision was automatically updated to reflect the committed changes.