diff options
| author | Chris Lattner <sabre@nondot.org> | 2010-10-01 05:36:09 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2010-10-01 05:36:09 +0000 |
| commit | a205055857fa3f945079d13787ba6c9e86682818 (patch) | |
| tree | 63c7d813112fb253acfb35b3e53ddd8331aac752 /llvm/lib/CodeGen/SelectionDAG | |
| parent | f419de0852852f068ebee271b29a6456b36b5fc2 (diff) | |
| download | bcm5719-llvm-a205055857fa3f945079d13787ba6c9e86682818.tar.gz bcm5719-llvm-a205055857fa3f945079d13787ba6c9e86682818.zip | |
fix rdar://8494845 + PR8244 - a miscompile exposed by my patch in r101350
llvm-svn: 115294
Diffstat (limited to 'llvm/lib/CodeGen/SelectionDAG')
| -rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp index be95c08e2fc..ede54e5ed23 100644 --- a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp @@ -4087,6 +4087,15 @@ SDValue DAGCombiner::ReduceLoadWidth(SDNode *N) { if ((N0.getValueType().getSizeInBits() & (EVTBits-1)) != 0) return SDValue(); } + + // If the shift amount is larger than the input type then we're not + // accessing any of the loaded bytes. If the load was a zextload/extload + // then the result of the shift+trunc is zero/undef (handled elsewhere). + // If the load was a sextload then the result is a splat of the sign bit + // of the extended byte. This is not worth optimizing for. + if (ShAmt >= VT.getSizeInBits()) + return SDValue(); + } } |

