diff options
author | Nadav Rotem <nrotem@apple.com> | 2012-08-13 18:52:44 +0000 |
---|---|---|
committer | Nadav Rotem <nrotem@apple.com> | 2012-08-13 18:52:44 +0000 |
commit | df8320313bf31c77d6dde4010235d82d871518cb (patch) | |
tree | 39a8276e2f223e42e37b773f56b2c2126c27b83c /lib/Target/ARM | |
parent | c586d268124f8801434cc0f68fe10cb3510c7ca6 (diff) | |
download | external_llvm-df8320313bf31c77d6dde4010235d82d871518cb.zip external_llvm-df8320313bf31c77d6dde4010235d82d871518cb.tar.gz external_llvm-df8320313bf31c77d6dde4010235d82d871518cb.tar.bz2 |
Do not optimize (or (and X,Y), Z) into BFI and other sequences if the AND ISDNode has more than one user.
rdar://11876519
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@161775 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/ARM')
-rw-r--r-- | lib/Target/ARM/ARMISelLowering.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/lib/Target/ARM/ARMISelLowering.cpp b/lib/Target/ARM/ARMISelLowering.cpp index 52bdd52..c66618a 100644 --- a/lib/Target/ARM/ARMISelLowering.cpp +++ b/lib/Target/ARM/ARMISelLowering.cpp @@ -7422,8 +7422,12 @@ static SDValue PerformORCombine(SDNode *N, return COR; } + + // The code below optimizes (or (and X, Y), Z). + // The AND operand needs to have a single user to make these optimizations + // profitable. SDValue N0 = N->getOperand(0); - if (N0.getOpcode() != ISD::AND) + if (N0.getOpcode() != ISD::AND || !N0.hasOneUse()) return SDValue(); SDValue N1 = N->getOperand(1); |