[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