This is an archive of the discontinued LLVM Phabricator instance.

[RISCVRVVInitUndef] Ignore tied use for partial undef register
ClosedPublic

Authored by reames on Jul 31 2023, 9:12 AM.

Details

Summary

The purpose of this code is to restrict overlap between source and destination registers. The tied input register is conceptually part of the destination. I can't see any reason why we need to prevent a partial undef tied source here, and skipping it reduces register pressure slightly.

Diff Detail

Event Timeline

reames created this revision.Jul 31 2023, 9:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 31 2023, 9:12 AM
reames requested review of this revision.Jul 31 2023, 9:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 31 2023, 9:12 AM

I think this makes sense to me, but I want to wait for @BeMg

BeMg accepted this revision.Jul 31 2023, 8:40 PM

LGTM

This revision is now accepted and ready to land.Jul 31 2023, 8:40 PM