khchen added a comment.

In D71387#1820995 <https://reviews.llvm.org/D71387#1820995>, @efriedma wrote:

> Okay.  Please let me know if you want me to review anything.

Hi all,
We had encoded the target-abi into module now, but I feel it does not make 
sense to 
support overwrite ABI option and datalayout in TargetMahcine/IR by target-abi 
module flag in IR.

So I think maybe passing the target-abi option by clang driver can make 
anything more simple, the only one limitation is users need to specific `-mabi` 
in below cases at the last command.

  clang -target riscv64-unknown-elf a.c -flto -march=rv64gc -mabi=lp64f -o a.o
  clang -target riscv64-unknown-elf b.c -flto -march=rv64gc -mabi=lp64f -o b.o
  clang -target riscv64-unknown-elf a.o b.o -flto -march=rv64gc -o foo




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71387

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to