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

Lu Weining via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 31 17:06:18 PDT 2023


SixWeining 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.
----------------
Better to sort alphabetically.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151794



More information about the llvm-commits mailing list