diff options
| author | Jim Laskey <jlaskey@mac.com> | 2006-09-26 08:14:06 +0000 |
|---|---|---|
| committer | Jim Laskey <jlaskey@mac.com> | 2006-09-26 08:14:06 +0000 |
| commit | 3dd1170616001e38b4dd899d5990734cea339663 (patch) | |
| tree | 70f0493f1d9b9e48799a46958f77c82d29e9b8a5 /lib/CodeGen/SelectionDAG/DAGCombiner.cpp | |
| parent | 172585b3aac4444e22d250a68e59bc03b8837ef4 (diff) | |
Wrong size for load
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@30610 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/SelectionDAG/DAGCombiner.cpp')
| -rw-r--r-- | lib/CodeGen/SelectionDAG/DAGCombiner.cpp | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp index 93f8db1237..9a78459ede 100644 --- a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp +++ b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp @@ -2644,6 +2644,8 @@ SDOperand DAGCombiner::visitLOAD(SDNode *N) { Chain.getOperand(1).getValueType() == N->getValueType(0)) return CombineTo(N, Chain.getOperand(1), Chain); + // We can only move the load if it has a user of it's chain result. Otherwise + // there is no place to attach it's old chain. if (CombinerAA && hasChainUsers(N)) { // Walk up chain skipping non-aliasing memory nodes. SDOperand BetterChain = FindBetterChain(N, Chain); @@ -3956,15 +3958,19 @@ SDOperand DAGCombiner::BuildUDIV(SDNode *N) { bool DAGCombiner::hasChainUsers(SDNode *Load) { // Don't even bother if the load only has one user (conservatively the value.) if (!Load->hasOneUse()) { - SDOperand Chain(Load, 1); + SDOperand Chain(Load, 1); // The load's chain result. + // For each user of the load. for (SDNode::use_iterator UI = Load->use_begin(), UE = Load->use_end(); UI != UE; ++UI) { + + // Chain will be the first operand. if ((*UI)->getOperand(0) == Chain) return true; } } + // No luck. return false; } @@ -4025,7 +4031,7 @@ void DAGCombiner::FindAliasInfo(SDNode *N, switch (N->getOpcode()) { case ISD::LOAD: Ptr = N->getOperand(1); - Size = MVT::getSizeInBits(N->getOperand(1).getValueType()) >> 3; + Size = MVT::getSizeInBits(N->getValueType(0)) >> 3; SrcValue = N->getOperand(2); break; case ISD::STORE: |