This is an archive of the discontinued LLVM Phabricator instance.

MachineSSAUpdater: Allow initialization with just a register class
ClosedPublic

Authored by nhaehnle on Aug 9 2020, 7:17 AM.

Details

Summary

The register class is required for inserting PHIs, but the "current
virtual register" isn't actually used for anything, so let's remove it
while we're at it.

Diff Detail

Event Timeline

nhaehnle created this revision.Aug 9 2020, 7:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 9 2020, 7:17 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
nhaehnle requested review of this revision.Aug 9 2020, 7:17 AM
arsenm accepted this revision.Aug 9 2020, 7:23 AM

This should probably support using an LLT/generic virtual register without a class, but that’s a separate change

This revision is now accepted and ready to land.Aug 9 2020, 7:23 AM
This revision was landed with ongoing or failed builds.Aug 21 2020, 2:05 PM
This revision was automatically updated to reflect the committed changes.