Print GEP offsets as signed values instead of unsigned values. On X86, this

prints:

getelementptr (int* %A, int -1)

as: "(A) - 4" instead of "(A) + 18446744073709551612", which makes the
assembler much happier.

This fixes test/Regression/CodeGen/X86/2005-02-14-IllegalAssembler.ll,
and Benchmarks/Prolangs-C/cdecl with LLC on X86.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@20183 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner 2005-02-14 21:40:26 +00:00
parent eb28797506
commit 27e192196f

View File

@ -78,10 +78,14 @@ void AsmPrinter::emitConstantValueOnly(const Constant *CV) {
// generate a symbolic expression for the byte address
const Constant *ptrVal = CE->getOperand(0);
std::vector<Value*> idxVec(CE->op_begin()+1, CE->op_end());
if (uint64_t Offset = TD.getIndexedOffset(ptrVal->getType(), idxVec)) {
O << "(";
if (int64_t Offset = TD.getIndexedOffset(ptrVal->getType(), idxVec)) {
if (Offset)
O << "(";
emitConstantValueOnly(ptrVal);
O << ") + " << Offset;
if (Offset > 0)
O << ") + " << Offset;
else if (Offset < 0)
O << ") - " << -Offset;
} else {
emitConstantValueOnly(ptrVal);
}