diff options
| author | Evgeniy Brevnov <ybrevnov@azul.com> | 2021-04-14 23:08:17 +0700 |
|---|---|---|
| committer | Evgeniy Brevnov <ybrevnov@azul.com> | 2021-04-19 12:13:22 +0700 |
| commit | 35e95c68176d599780c3907afe6c0c4c5162672f (patch) | |
| tree | e5ffffd130f9af941864a4a15274deb4b35b2f05 /clang/unittests/Frontend/CompilerInvocationTest.cpp | |
| parent | 27edaee84e3ea4d160f742db0b4a04e236c4e26e (diff) | |
| download | llvm-35e95c68176d599780c3907afe6c0c4c5162672f.zip llvm-35e95c68176d599780c3907afe6c0c4c5162672f.tar.gz llvm-35e95c68176d599780c3907afe6c0c4c5162672f.tar.bz2 | |
[CVP] processCallSite returns wrong status
Recently processMinMaxIntrinsic has been added and we started to observe a number of analysis get invalidated after CVP. The problem is CVP conservatively returns 'true' even if there were no modifications to IR. I found one more place besides processMinMaxIntrinsic which has the same problem. I think processMinMaxIntrinsic and similar should better have boolean return status to prevent similar issue reappear in future.
Reviewed By: lebedev.ri
Differential Revision: https://reviews.llvm.org/D100538
Diffstat (limited to 'clang/unittests/Frontend/CompilerInvocationTest.cpp')
0 files changed, 0 insertions, 0 deletions