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

Lu Weining via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 31 23:18:35 PDT 2022


SixWeining added a comment.

In D136436#3898392 <https://reviews.llvm.org/D136436#3898392>, @tangyouling wrote:

> Is it acceptable to add the `non-prefix $`? if not, an alternative fix for the build failure is to add the `prefix $` in sanitizer_syscall_linux_loongarch64.inc

People have different opinions. This may need long time to discuss and decide (maybe in a broader place). So currently it's better to modify `sanitizer_syscall_linux_loongarch64.inc` I think.


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