diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2016-06-06 16:56:57 +0000 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2016-06-06 16:56:57 +0000 |
| commit | 6a333c3ed9006e9140e7b29dfb36b4fe37825eaf (patch) | |
| tree | 6141e786bcad44a1f7d8495f1cc03a9b5588c4eb /clang/lib/StaticAnalyzer/Checkers/MPI-Checker/MPIBugReporter.cpp | |
| parent | 83ccd1a994b21c8f98fe4b44d56dcba7cea4b5c6 (diff) | |
| download | bcm5719-llvm-6a333c3ed9006e9140e7b29dfb36b4fe37825eaf.tar.gz bcm5719-llvm-6a333c3ed9006e9140e7b29dfb36b4fe37825eaf.zip | |
[InstCombine] limit icmp transform to ConstantInt (PR28011)
In r271810 ( http://reviews.llvm.org/rL271810 ), I loosened the check
above this to work for any Constant rather than ConstantInt. AFAICT,
that part makes sense if we can determine that the shrunken/extended
constant remained equal. But it doesn't make sense for this later
transform where we assume that the constant DID change.
This could assert for a ConstantExpr:
https://llvm.org/bugs/show_bug.cgi?id=28011
And it could be wrong for a vector as shown in the added regression test.
llvm-svn: 271908
Diffstat (limited to 'clang/lib/StaticAnalyzer/Checkers/MPI-Checker/MPIBugReporter.cpp')
0 files changed, 0 insertions, 0 deletions
