[clang] [llvm] [clang][RISCV] Change default abi with f extension but without d extension (PR #73489)

https://github.com/jacquesguan closed https://github.com/llvm/llvm-project/pull/73489 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [clang][RISCV] Change default abi with f extension but without d extension (PR #73489)

https://github.com/asb approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/73489 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [clang][RISCV] Change default abi with f extension but without d extension (PR #73489)

asb wrote: > > I think the conclusion from the LLVM sync-up call was that everyone happy > > to move in this direction, so please add the release note and we can do a > > final review. Thanks! > > Done, added release note. Thanks! Sorry I wasn't specific about this, but we need a Clang releas

[clang] [llvm] [clang][RISCV] Change default abi with f extension but without d extension (PR #73489)

asb wrote: I think the conclusion from the LLVM sync-up call was that everyone happy to move in this direction, so please add the release note and we can do a final review. Thanks! https://github.com/llvm/llvm-project/pull/73489 ___ cfe-commits maili

[clang] [llvm] [clang][RISCV] Change default abi with f extension but without d extension (PR #73489)

wangpc-pp wrote: As @kito-cheng has pointed out, we should fix multilib issue before landing this. It seems we don't generate multilib for ilp32f and lp64f in riscv-gnu-toolchain? https://github.com/llvm/llvm-project/pull/73489 ___ cfe-commits mailin

[clang] [llvm] [clang][RISCV] Change default abi with f extension but without d extension (PR #73489)

https://github.com/benshi001 approved this pull request. https://github.com/llvm/llvm-project/pull/73489 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [clang][RISCV] Change default abi with f extension but without d extension (PR #73489)

benshi001 wrote: I think this is reasonable, even I myself made such a change months ago in Phabracator, but finally not approved. https://github.com/llvm/llvm-project/pull/73489 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.l