[PATCH] D151794: [LoongArch] Add sign extension for i32 parameters and returns.

[Limin Zhang via Phabricator via llvm-commits]

Ami-zhang added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetLibraryInfo.h:415
     }
-    // Mips and riscv64, on the other hand, needs signext on i32 parameters
-    // corresponding to both signed and unsigned ints.
-    if (T.isMIPS() || T.isRISCV64()) {
+    // Mips, riscv64, and LoongArch, on the other hand, need signext on i32
+    // parameters corresponding to both signed and unsigned ints.
----------------
SixWeining wrote:
> Better to sort alphabetically.
Ok.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151794