diff options
| author | Owen Anderson <resistor@mac.com> | 2013-04-23 18:09:28 +0000 |
|---|---|---|
| committer | Owen Anderson <resistor@mac.com> | 2013-04-23 18:09:28 +0000 |
| commit | ed5707baf9be36c0d06c278f589b6c55db8b96c0 (patch) | |
| tree | adc2b1c64d4a02d52fc592173f1171279cb1d6f4 /lib/CodeGen/SelectionDAG/DAGCombiner.cpp | |
| parent | 3579a29dba8fcc7285a3a5bec4512b4ca0a91ab3 (diff) | |
| download | llvm-ed5707baf9be36c0d06c278f589b6c55db8b96c0.tar.gz llvm-ed5707baf9be36c0d06c278f589b6c55db8b96c0.tar.bz2 llvm-ed5707baf9be36c0d06c278f589b6c55db8b96c0.tar.xz | |
DAGCombine should not aggressively fold SEXT(VSETCC(...)) into a wider VSETCC without first checking the target's vector boolean contents.
This exposed an issue with PowerPC AltiVec where it appears it was setting the wrong vector boolean contents. The included change
fixes the PowerPC tests, and was OK'd by Hal.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@180129 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/SelectionDAG/DAGCombiner.cpp')
| -rw-r--r-- | lib/CodeGen/SelectionDAG/DAGCombiner.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp index 3b823ad9b6..f67cd3e270 100644 --- a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp +++ b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp @@ -4451,7 +4451,9 @@ SDValue DAGCombiner::visitSIGN_EXTEND(SDNode *N) { if (N0.getOpcode() == ISD::SETCC) { // sext(setcc) -> sext_in_reg(vsetcc) for vectors. // Only do this before legalize for now. - if (VT.isVector() && !LegalOperations) { + if (VT.isVector() && !LegalOperations && + TLI.getBooleanContents(true) == + TargetLowering::ZeroOrNegativeOneBooleanContent) { EVT N0VT = N0.getOperand(0).getValueType(); // On some architectures (such as SSE/NEON/etc) the SETCC result type is // of the same size as the compared operands. Only optimize sext(setcc()) |