[PATCH] D128572: [LoongArch] Add codegen support for division operations
Xi Ruoyao via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 28 03:47:09 PDT 2022
xry111 added a comment.
In D128572#3614246 <https://reviews.llvm.org/D128572#3614246>, @xen0n wrote:
> LGTM, thanks!
>
> (I've checked harder and apparently the weird "erratum" is in fact a wart carried over from MIPS. The MIPS ISA manual contain the very same wording regarding non-canonical inputs to 32-bit division/modulus operations. Let's just hope this gets fixed in future models...)
I guess the ALU of 3A5000 is not deviated too much from 3A4000, and it limits LA v1.00 to keep such division results undefined. It's usually a good thing to replace components one by one (instead of replacing many components in one shot) in a complex product like CPU though.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128572/new/
https://reviews.llvm.org/D128572
More information about the llvm-commits
mailing list