diff options
author | Eli Friedman <eli.friedman@gmail.com> | 2016-06-11 21:48:25 +0000 |
---|---|---|
committer | Eli Friedman <eli.friedman@gmail.com> | 2016-06-11 21:48:25 +0000 |
commit | f1da33e4d3fc9797c7fb4f9ea681a2bc2254c281 (patch) | |
tree | 32d783ef43cbe5b7a1a6f91ba40f69f6cdb22213 /clang/www/builtins.py | |
parent | 2b7c02a04f465c298d008ebf8b7cc616e3d3db89 (diff) | |
download | bcm5719-llvm-f1da33e4d3fc9797c7fb4f9ea681a2bc2254c281.tar.gz bcm5719-llvm-f1da33e4d3fc9797c7fb4f9ea681a2bc2254c281.zip |
[LICM] Make isGuaranteedToExecute more accurate.
Summary:
Make isGuaranteedToExecute use the
isGuaranteedToTransferExecutionToSuccessor helper, and make that helper
a bit more accurate.
There's a potential performance impact here from assuming that arbitrary
calls might not return. This probably has little impact on loads and
stores to a pointer because most things alias analysis can reason about
are dereferenceable anyway. The other impacts, like less aggressive
hoisting of sdiv by a variable and less aggressive hoisting around
volatile memory operations, are unlikely to matter for real code.
This also impacts SCEV, which uses the same helper. It's a minor
improvement there because we can tell that, for example, memcpy always
returns normally. Strictly speaking, it's also introducing
a bug, but it's not any worse than everywhere else we assume readonly
functions terminate.
Fixes http://llvm.org/PR27857.
Reviewers: hfinkel, reames, chandlerc, sanjoy
Subscribers: broune, llvm-commits
Differential Revision: http://reviews.llvm.org/D21167
llvm-svn: 272489
Diffstat (limited to 'clang/www/builtins.py')
0 files changed, 0 insertions, 0 deletions