[PATCH] D21739: [TLI] Add functions determining if int parameters/returns should be zeroext/signext.

Simon Dardis via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 10 11:11:00 PST 2016


sdardis added inline comments.


================
Comment at: lib/Analysis/TargetLibraryInfo.cpp:428
+  // Mips, on the other hand, needs signext on i32 parameters corresponding
+  // to both signed and unsigned ints.
+  if (T.getArch() == Triple::mips || T.getArch() == Triple::mipsel ||
----------------
hfinkel wrote:
> To be clear, they sign extend even unsigned numbers?
Yes. The N64 and N32 ABIs requires that 32 bit integer parameters are sign extended, even unsigned numbers.


Repository:
  rL LLVM

https://reviews.llvm.org/D21739





More information about the llvm-commits mailing list