[PATCH] D98532: [M68k] Use fixed asm string for MxPseudo instructions

Min-Yih Hsu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 17 12:00:19 PDT 2021


myhsu added a comment.

In D98532#2629887 <https://reviews.llvm.org/D98532#2629887>, @RKSimon wrote:

> In D98532#2627487 <https://reviews.llvm.org/D98532#2627487>, @myhsu wrote:
>
>> In D98532#2626683 <https://reviews.llvm.org/D98532#2626683>, @RKSimon wrote:
>>
>>> @myhsu Would we be better off adding non-empty asm strings to all uses?
>>
>> Do you mean setting non-empty asm string by default on `MxInst`? I feel like the non-empty string requirement imposed by parser table can help us catch mistakes if we forget to set assembly string for an instruction (in the future), so I'm incline not to do that
>
> I meant just stop MxPseudo taking a default string so all instances include a useful asm comment string.

Ah, using the asm string as comments (for MxPseudo instances), I see. But I don't think these pseudo instructions will ever be printed/parsed to/from assembly. From what I understand they're only syntax sugar for codegen (unlike some of the targets that have assembler pseudo instructions). If that's the case, I don't think it's really necessary to put meaningful string in the assembly string. If we want to add comments, I prefer to put in the TG code.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98532/new/

https://reviews.llvm.org/D98532



More information about the llvm-commits mailing list