diff options
| author | QingShan Zhang <qshanz@cn.ibm.com> | 2020-05-11 02:41:10 +0000 |
|---|---|---|
| committer | QingShan Zhang <qshanz@cn.ibm.com> | 2020-05-11 02:41:10 +0000 |
| commit | 3c44c441db0f8d7e210806b5b221cd9ed66f2d7b (patch) | |
| tree | e2af4d8b7525e4e81c794483094b0161929a4c70 /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | 8e63f35f86bd9b8d014cd3488c09465bba34c1f7 (diff) | |
| download | llvm-3c44c441db0f8d7e210806b5b221cd9ed66f2d7b.zip llvm-3c44c441db0f8d7e210806b5b221cd9ed66f2d7b.tar.gz llvm-3c44c441db0f8d7e210806b5b221cd9ed66f2d7b.tar.bz2 | |
[DAGCombine] Remove the getNegatibleCost to avoid the out of sync with getNegatedExpression
We have the getNegatibleCost/getNegatedExpression to evaluate the cost and negate the expression.
However, during negating the expression, the cost might change as we are changing the DAG,
and then, hit the assertion if we negated the wrong expression as the cost is not trustful anymore.
This patch is target to remove the getNegatibleCost to avoid the out of sync with getNegatedExpression,
and check the cost during negating the expression. It also reduce the duplicated code between
getNegatibleCost and getNegatedExpression. And fix the crash for the test in D76638
Reviewed By: RKSimon, spatel
Differential Revision: https://reviews.llvm.org/D77319
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions