This is an archive of the discontinued LLVM Phabricator instance.

[SystemZ][z/OS] Enforce prefix-less registers in SystemZAsmParser for the HLASM dialect.
ClosedPublic

Authored by anirudhp on Apr 30 2021, 2:12 PM.

Details

Summary
  • Previously, https://reviews.llvm.org/D101308 removed prefixes from register while printing them out. This was especially needed for inline asm statements which used input/output operands.
  • However, the backend SystemZAsmParser, accepts both prefixed registers and prefix-less registers as part of its implementation
  • This patch aims to change that by ensuring that prefixed registers are only allowed for the ATT dialect.

Diff Detail

Event Timeline

anirudhp created this revision.Apr 30 2021, 2:12 PM
anirudhp requested review of this revision.Apr 30 2021, 2:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 30 2021, 2:12 PM
uweigand accepted this revision.May 3 2021, 4:20 AM

LGTM, thanks!

This revision is now accepted and ready to land.May 3 2021, 4:20 AM