diff options
author | Owen Anderson <resistor@mac.com> | 2013-02-05 06:25:30 +0000 |
---|---|---|
committer | Owen Anderson <resistor@mac.com> | 2013-02-05 06:25:30 +0000 |
commit | 429f7ef0c116c0504052b9a6655ef4d973177e9d (patch) | |
tree | 7eab6bb6277c7a24d30b863d288e44f7096b4980 /lib/CodeGen | |
parent | a2de37c897556fbd1f94a3ed84ad27accd8f8deb (diff) | |
download | external_llvm-429f7ef0c116c0504052b9a6655ef4d973177e9d.zip external_llvm-429f7ef0c116c0504052b9a6655ef4d973177e9d.tar.gz external_llvm-429f7ef0c116c0504052b9a6655ef4d973177e9d.tar.bz2 |
When the target-independent DAGCombiner inferred a higher alignment for a load,
it would replace the load with one with the higher alignment. However, it did
not place the new load in the worklist, which prevented later DAG combines in
the same phase (for example, target-specific combines) from ever seeing it.
This patch corrects that oversight, and updates some tests whose output changed
due to slightly different DAGCombine outputs.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@174343 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen')
-rw-r--r-- | lib/CodeGen/SelectionDAG/DAGCombiner.cpp | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp index 79ec227..39a8e82 100644 --- a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp +++ b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp @@ -7199,12 +7199,15 @@ SDValue DAGCombiner::visitLOAD(SDNode *N) { // Try to infer better alignment information than the load already has. if (OptLevel != CodeGenOpt::None && LD->isUnindexed()) { if (unsigned Align = DAG.InferPtrAlignment(Ptr)) { - if (Align > LD->getAlignment()) - return DAG.getExtLoad(LD->getExtensionType(), N->getDebugLoc(), + if (Align > LD->getAlignment()) { + SDValue NewLoad = + DAG.getExtLoad(LD->getExtensionType(), N->getDebugLoc(), LD->getValueType(0), Chain, Ptr, LD->getPointerInfo(), LD->getMemoryVT(), LD->isVolatile(), LD->isNonTemporal(), Align); + return CombineTo(N, NewLoad, SDValue(NewLoad.getNode(), 1), true); + } } } |