[PATCH] D134638: [Clang][LoongArch] Add inline asm support for constraints k/m/ZB/ZC

Thu Sep 29 04:56:01 PDT 2022

SixWeining added a comment.

In D134638#3823379 <https://reviews.llvm.org/D134638#3823379>, @xen0n wrote:

> The previous test results (with some of my WIP patches but unrelated to this) before the fix:
>
>   Failed Tests (6):
>     LLVM :: CodeGen/AArch64/GlobalISel/irtranslator-inline-asm.ll
>     LLVM :: CodeGen/AMDGPU/GlobalISel/irtranslator-inline-asm.ll
>     LLVM :: CodeGen/Generic/vector.ll
>     LLVM :: CodeGen/PowerPC/2007-11-19-VectorSplitting.ll
>     LLVM :: CodeGen/X86/callbr-asm-kill.mir
>     LLVM :: DebugInfo/Generic/missing-abstract-variable.ll
>
> Sure enough it was caught by my native build with all targets enabled, anyway it's important to at least enable the "big" targets such as X86 and AArch64 when configuring the local dev env. ;-)

Thanks for the tip. Just now I enabled all official targets and LoongArch and find 2 more fail tests (for same reason) for AArch64 and AMDGPU. I have fixed them and updated the patch.

Sorry again...

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134638