[PATCH] D102839: [RISCV][Clang] Add -mno-div option to disable hardware int division

Luís Marques via Phabricator via cfe-commits cfe-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 cfe-commits mailing list