[PATCH] D113959: [llvm][RISC-V] Use floating-point ABI by default if possible
Jessica Clarke via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 9 07:46:12 PST 2021
jrtc27 added a comment.
FYI the llvmlite PR to set MCOptions.ABIName got merged.
I continue to believe that this should not be necessary from an LLVM API consumer perspective. Anything not setting the ABI is broken and should be fixed, and it's not our problem that they don't use our API properly or exist in an ecosystem that makes it difficult to patch them. If anything I would rather there be //no// default in the backend, only in the Clang frontend, but the main reason not to do that is it's annoying for writing tests where most don't care what floating-point ABI is in use.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113959/new/
https://reviews.llvm.org/D113959
More information about the llvm-commits
mailing list