This is an archive of the discontinued LLVM Phabricator instance.

[LoongArch] Add missing dollar prefix to register name in InstPrinter
ClosedPublic

Authored by SixWeining on Feb 15 2022, 12:53 AM.

Details

Summary

This patch adds a '$' prefix to register name in InstPrinter that I missed in initial patches.

Diff Detail

Event Timeline

SixWeining created this revision.Feb 15 2022, 12:53 AM
SixWeining requested review of this revision.Feb 15 2022, 12:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 15 2022, 12:53 AM
xen0n accepted this revision.Feb 15 2022, 1:47 AM

LGTM, that's a bit hard for me to catch!

(me being way too familiar with RISC-V assembly, where unprefixed register names are okay to use)

This revision is now accepted and ready to land.Feb 15 2022, 1:47 AM
This revision was landed with ongoing or failed builds.Feb 17 2022, 5:57 PM
This revision was automatically updated to reflect the committed changes.