[PATCH] D128572: [LoongArch] Add codegen support for division operations

[WÁNG Xuěruì via Phabricator via llvm-commits]

xen0n added a comment.

In D128572#3609943 <https://reviews.llvm.org/D128572#3609943>, @SixWeining wrote:

> riscv and aarch64 do not check and I'm not sure it is because their ISAs define zero-divide having an fixed result while Mips and LoongArch do not.

Also, technically LoongArch could be revised to produce similarly defined results as RISC-V or AArch64, because anything is compatible with "undefined" so we won't break compatibility with the v1.00 ISA semantics. You could discuss this with your hardware team too if you want; IMO the RISC-V approach seems very reasonable, for example.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128572