[PATCH] D136436: [Clang][LoongArch] Add register alias handling without `$` prefix

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 24 20:37:36 PDT 2022


MaskRay added a comment.

In D136436#3876684 <https://reviews.llvm.org/D136436#3876684>, @SixWeining wrote:

> In D136436#3873987 <https://reviews.llvm.org/D136436#3873987>, @xen0n wrote:
>
>> In D136436#3873949 <https://reviews.llvm.org/D136436#3873949>, @SixWeining wrote:
>>
>>> How about the asm code in `.s`? Do we need to support `addi.d a0, a1, a2`?
>>
>> For the assembler part of this support, I think we need to coordinate with the GNU toolchain maintainers of LoongArch port (Chenghua, Zhensong and Lulu I think). Maybe raising an issue on the LoongArch documentation repo <https://github.com/loongson/LoongArch-Documentation> would help.
>>
>> For the GCC part, consistency is of course welcomed, and I think the Loongson maintainers or @xry111 could just submit the respective support.
>
> Well. To be honest, I'd like to keep only one form but not both. Seems other archs only support one form? If we support both, will it make people confused?  Is mixed form allowed (`fmadd.d fa0, $fa0, f0, $fcc0`) ?

Having just one form will be nice.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136436



More information about the cfe-commits mailing list