6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-01-19 04:32:19 +00:00
Code Issues Projects Releases Wiki Activity

Re-write the address propagation code for pre-indexed loads/stores to take into account some previously misssed cases (PRE_DEC addressing mode, the offset and base address are swapped, etc). This should fix PR15581.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@180609 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Silviu Baranga 2013-04-26 15:52:24 +00:00
parent 069a4a9583
commit 730a570c8c
1 changed files with 29 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

43
lib/CodeGen/SelectionDAG/DAGCombiner.cpp
View File

@ -7152,25 +7152,40 @@ bool DAGCombiner::CombineToPreIndexedLoadStore(SDNode *N) {
assert(OtherUses[i]->getOperand(!OffsetIdx).getNode() == assert(OtherUses[i]->getOperand(!OffsetIdx).getNode() ==
BasePtr.getNode() && "Expected BasePtr operand"); BasePtr.getNode() && "Expected BasePtr operand");
APInt OV = // We need to replace ptr0 in the following expression:
cast<ConstantSDNode>(Offset)->getAPIntValue(); // x0 * offset0 + y0 * ptr0 = t0
if (AM == ISD::PRE_DEC) // knowing that
OV = -OV; // x1 * offset1 + y1 * ptr0 = t1 (the indexed load/store)
//
// where x0, x1, y0 and y1 in {-1, 1} are given by the types of the
// indexed load/store and the expresion that needs to be re-written.
//
// Therefore, we have:
// t0 = (x0 * offset0 - x1 * y0 * y1 *offset1) + (y0 * y1) * t1
ConstantSDNode *CN = ConstantSDNode *CN =
cast<ConstantSDNode>(OtherUses[i]->getOperand(OffsetIdx)); cast<ConstantSDNode>(OtherUses[i]->getOperand(OffsetIdx));
APInt CNV = CN->getAPIntValue(); int X0, X1, Y0, Y1;
if (OtherUses[i]->getOpcode() == ISD::SUB && OffsetIdx == 1) APInt Offset0 = CN->getAPIntValue();
CNV += OV; APInt Offset1 = cast<ConstantSDNode>(Offset)->getAPIntValue();
else
CNV -= OV;
SDValue NewOp1 = Result.getValue(isLoad ? 1 : 0); X0 = (OtherUses[i]->getOpcode() == ISD::SUB && OffsetIdx == 1) ? -1 : 1;
SDValue NewOp2 = DAG.getConstant(CNV, CN->getValueType(0)); Y0 = (OtherUses[i]->getOpcode() == ISD::SUB && OffsetIdx == 0) ? -1 : 1;
if (OffsetIdx == 0) X1 = (AM == ISD::PRE_DEC && !Swapped) ? -1 : 1;
std::swap(NewOp1, NewOp2); Y1 = (AM == ISD::PRE_DEC && Swapped) ? -1 : 1;
SDValue NewUse = DAG.getNode(OtherUses[i]->getOpcode(), unsigned Opcode = (Y0 * Y1 < 0) ? ISD::SUB : ISD::ADD;
APInt CNV = Offset0;
if (X0 < 0) CNV = -CNV;
if (X1 * Y0 * Y1 < 0) CNV = CNV + Offset1;
else CNV = CNV - Offset1;
// We can now generate the new expression.
SDValue NewOp1 = DAG.getConstant(CNV, CN->getValueType(0));
SDValue NewOp2 = Result.getValue(isLoad ? 1 : 0);
SDValue NewUse = DAG.getNode(Opcode,
OtherUses[i]->getDebugLoc(), OtherUses[i]->getDebugLoc(),
OtherUses[i]->getValueType(0), NewOp1, NewOp2); OtherUses[i]->getValueType(0), NewOp1, NewOp2);
DAG.ReplaceAllUsesOfValueWith(SDValue(OtherUses[i], 0), NewUse); DAG.ReplaceAllUsesOfValueWith(SDValue(OtherUses[i], 0), NewUse);