[PowerPC] Print all inline-asm consts as signed numbers
authorHal Finkel <hfinkel@anl.gov>
Wed, 3 Dec 2014 09:37:50 +0000 (09:37 +0000)
committerHal Finkel <hfinkel@anl.gov>
Wed, 3 Dec 2014 09:37:50 +0000 (09:37 +0000)
commitcf988bca67249f3146a380df888514d1a2b2a7ee
tree17c5f5de8aab608ce4b3100749a5d979227fd1ba
parent10cae8e35228ddc72f6f3f79dcc6b891b9ee2042
[PowerPC] Print all inline-asm consts as signed numbers

Almost all immediates in PowerPC assembly (both 32-bit and 64-bit) are signed
numbers, and it is important that we print them as such. To make sure that
happens, we change PPCTargetLowering::LowerAsmOperandForConstraint so that it
does all intermediate checks on a signed-extended int64_t value, and then
creates the resulting target constant using MVT::i64. This will ensure that all
negative values are printed as negative values (mirroring what is done in other
backends to achieve the same sign-extension effect).

This came up in the context of inline assembly like this:
  "add%I2   %0,%0,%2", ..., "Ir"(-1ll)
where we used to print:
  addi   3,3,4294967295
and gcc would print:
  addi   3,3,-1
and gas accepts both forms, but our builtin assembler (correctly) does not. Now
we print -1 like gcc does.

While here, I replaced a bunch of custom integer checks with isInt<16> and
friends from MathExtras.h.

Thanks to Paul Hargrove for the bug report.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@223220 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/PowerPC/PPCISelLowering.cpp
test/CodeGen/PowerPC/ia-neg-const.ll [new file with mode: 0644]