diff options
author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2017-11-18 18:33:07 +0000 |
---|---|---|
committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2017-11-18 18:33:07 +0000 |
commit | c9bc55a08d62c64ae478a1c4ac58087ac2f597bd (patch) | |
tree | b2dc6ec81bd75b365c11a6dd4d0f52c097a0325b /llvm/lib | |
parent | 2a266a343fece3f04709794efcb3929c542d2762 (diff) | |
download | bcm5719-llvm-c9bc55a08d62c64ae478a1c4ac58087ac2f597bd.tar.gz bcm5719-llvm-c9bc55a08d62c64ae478a1c4ac58087ac2f597bd.zip |
[X86] Add todo comment for TRUNC(SUB(X,C)) -> SUB(TRUNC(X),C')
As discussed on PR35295, but it causes regressions in combineSubToSubus which need to be addressed first
llvm-svn: 318594
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/Target/X86/X86ISelLowering.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index 9474c59e9f6..f5daec99c83 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/lib/Target/X86/X86ISelLowering.cpp @@ -34482,6 +34482,7 @@ static SDValue combineTruncatedArithmetic(SDNode *N, SelectionDAG &DAG, return TruncateArithmetic(Src.getOperand(0), Src.getOperand(1)); LLVM_FALLTHROUGH; case ISD::ADD: { + // TODO: ISD::SUB should be here but interferes with combineSubToSubus. SDValue Op0 = Src.getOperand(0); SDValue Op1 = Src.getOperand(1); if (TLI.isOperationLegal(Opcode, VT) && |