diff options
author | Roman Lebedev <lebedev.ri@gmail.com> | 2019-09-24 16:10:50 +0000 |
---|---|---|
committer | Roman Lebedev <lebedev.ri@gmail.com> | 2019-09-24 16:10:50 +0000 |
commit | 45fd1e9d503c2f300a7d52ca2c468131c381dceb (patch) | |
tree | e76a0a7920d7a81b53ced05ad5219dd78d7f08f3 /lldb/packages/Python/lldbsuite/test/python_api/sbstructureddata | |
parent | 5b881f356cfecd2849fd5063c1fd816e7bbb35f7 (diff) | |
download | bcm5719-llvm-45fd1e9d503c2f300a7d52ca2c468131c381dceb.tar.gz bcm5719-llvm-45fd1e9d503c2f300a7d52ca2c468131c381dceb.zip |
[InstCombine] (a+b) < a && (a+b) != 0 -> (0-b) < a iff a/b != 0 (PR43259)
Summary:
This is again motivated by D67122 sanitizer check enhancement.
That patch seemingly worsens `-fsanitize=pointer-overflow`
overhead from 25% to 50%, which strongly implies missing folds.
For
```
#include <cassert>
char* test(char& base, signed long offset) {
__builtin_assume(offset < 0);
return &base + offset;
}
```
We produce
https://godbolt.org/z/r40U47
and again those two icmp's can be merged:
```
Name: 0
Pre: C != 0
%adjusted = add i8 %base, C
%not_null = icmp ne i8 %adjusted, 0
%no_underflow = icmp ult i8 %adjusted, %base
%r = and i1 %not_null, %no_underflow
=>
%neg_offset = sub i8 0, C
%r = icmp ugt i8 %base, %neg_offset
```
https://rise4fun.com/Alive/ALap
https://rise4fun.com/Alive/slnN
There are 3 other variants of this pattern,
i believe they all will go into InstSimplify.
https://bugs.llvm.org/show_bug.cgi?id=43259
Reviewers: spatel, xbolva00, nikic
Reviewed By: spatel
Subscribers: efriedma, hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D67849
llvm-svn: 372768
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/python_api/sbstructureddata')
0 files changed, 0 insertions, 0 deletions