[PATCH] D102839: [RISCV][Clang] Add -mno-div option to disable hardware int division
Luís Marques via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 27 03:49:51 PDT 2021
luismarques added a comment.
In D102839#2784275 <https://reviews.llvm.org/D102839#2784275>, @ksyx wrote:
> Thanks for mentioning that! Now, I changed the effect of `no-div` option into choosing a proper extension and implemented the `Zmmul` subextension. Is this solution acceptable or are there anything need further changes?
It seems like the community is quickly converging on just using the ISA string with Zmmul, and not using no-div. While being compatible with the GNU tools is nice, if they are planning on dropping support for no-div soonish then we probably shouldn't add support for it. IMO, Zmmul should be a separate patch and presumably (at the moment) be gated by `-menable-experimental-extensions`.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102839/new/
https://reviews.llvm.org/D102839
More information about the llvm-commits
mailing list