Page MenuHomePhabricator

[SystemZ] Bugfix for the 'N' code for inline asm operand.
ClosedPublic

Authored by jonpa on Jul 10 2021, 2:37 AM.

Details

Summary

Don't use a local MachineOperand copy in SystemZAsmPrinter::PrintAsmOperand() and change the register as it may break the MRI tracking of register uses. Use an MCOperand instead.

Diff Detail

Event Timeline

jonpa created this revision.Jul 10 2021, 2:37 AM
jonpa requested review of this revision.Jul 10 2021, 2:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 10 2021, 2:37 AM

Thanks, I can confirm this fixes the crash that I saw and does not regress the build or boot in other ways.

uweigand accepted this revision.Jul 12 2021, 1:55 AM

LGTM, thanks!

This revision is now accepted and ready to land.Jul 12 2021, 1:55 AM
This revision was landed with ongoing or failed builds.Jul 12 2021, 6:16 AM
This revision was automatically updated to reflect the committed changes.