diff options
author | Craig Topper <craig.topper@intel.com> | 2017-07-04 06:50:48 +0000 |
---|---|---|
committer | Craig Topper <craig.topper@intel.com> | 2017-07-04 06:50:48 +0000 |
commit | 0f746c2793aa32273fc23c6ac156356195caed94 (patch) | |
tree | cfc4cbf8fdd6cf85628a895b220d4357e08d0972 /llvm/lib/Transforms | |
parent | 872d75056092025b3bc1025bdf4417011bdb2844 (diff) | |
download | bcm5719-llvm-0f746c2793aa32273fc23c6ac156356195caed94.tar.gz bcm5719-llvm-0f746c2793aa32273fc23c6ac156356195caed94.zip |
[InstCombine] Add TODOs for a couple things that should maybe be in InstSimplify instead. NFC
llvm-svn: 307065
Diffstat (limited to 'llvm/lib/Transforms')
-rw-r--r-- | llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp b/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp index 3770021de10..394a55a1294 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp +++ b/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp @@ -1965,9 +1965,10 @@ Instruction *InstCombiner::visitCallInst(CallInst &CI) { Value *IIOperand = II->getArgOperand(0); Value *X = nullptr; + // TODO should this be in InstSimplify? // bswap(bswap(x)) -> x if (match(IIOperand, m_BSwap(m_Value(X)))) - return replaceInstUsesWith(CI, X); + return replaceInstUsesWith(CI, X); // bswap(trunc(bswap(x))) -> trunc(lshr(x, c)) if (match(IIOperand, m_Trunc(m_BSwap(m_Value(X))))) { @@ -1984,6 +1985,7 @@ Instruction *InstCombiner::visitCallInst(CallInst &CI) { Value *IIOperand = II->getArgOperand(0); Value *X = nullptr; + // TODO should this be in InstSimplify? // bitreverse(bitreverse(x)) -> x if (match(IIOperand, m_BitReverse(m_Value(X)))) return replaceInstUsesWith(CI, X); |