[PATCH] D136436: [Clang][LoongArch] Add register alias handling without `$` prefix
Lu Weining via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 21 20:53:23 PDT 2022
SixWeining added a comment.
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, $f0`) ?
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