[PATCH] D102582: [RISCV] Report an error when ABI mismatch with target-abi module flag.

[Zakk Chen via Phabricator via llvm-commits]

khchen added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/module-target-abi-tests.ll:5
+; RUN: cat %s > %t.emptyabi
+; RUN: echo '!0 = !{i32 1, !"target-abi", !""}' >> %t.emptyabi
+; RUN: llc -mtriple=riscv32 < %t.emptyabi -o /dev/null
----------------
khchen wrote:
> luismarques wrote:
> > Is this something that we are handling in general, having such flag without a value?
> Good cached, in general the target-abi is not empty, I updated the current implementation, thanks!
@luismarques 

Sorry, I forget that the empty target-abi are coming from some clang cc1 tests. 
They are missing -target-abi option in clang cc1 so target-abi module flag is empty.
 
```
CodeGen/RISCV/riscv-atomics.c
CodeGen/RISCV/riscv-inline-asm-rvv.c
CodeGen/RISCV/riscv-inline-asm-xsfvfhbfmin.c
CodeGen/RISCV/riscv-inline-asm.c
```

Maybe we need to calculate the default target-abi if it's empty? or handle empty target-abi in the backend?



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102582