diff options
author | Philip Reames <listmail@philipreames.com> | 2019-11-06 15:30:47 -0800 |
---|---|---|
committer | Philip Reames <listmail@philipreames.com> | 2019-11-06 15:41:57 -0800 |
commit | 8748be7750920b7bd669a47be72a81ee7f1d305c (patch) | |
tree | 69947b64b777fc5422b60019424b5c00787e2851 /llvm/lib/Transforms/Scalar/IndVarSimplify.cpp | |
parent | 20cbb6cdf8b33dd3732b9ef8702e7946b6d8c739 (diff) | |
download | bcm5719-llvm-8748be7750920b7bd669a47be72a81ee7f1d305c.tar.gz bcm5719-llvm-8748be7750920b7bd669a47be72a81ee7f1d305c.zip |
[LoopPred] Enable new transformation by default
The basic idea of the transform is to convert variant loop exit conditions into invariant exit conditions by changing the iteration on which the exit is taken when we know that the trip count is unobservable. See the original patch which introduced the code for a more complete explanation.
The individual parts of this have been reviewed, the result has been fuzzed, and then further analyzed by hand, but despite all of that, I will not be suprised to see breakage here. If you see problems, please don't hesitate to revert - though please do provide a test case. The most likely class of issues are latent SCEV bugs and without a reduced test case, I'll be essentially stuck on reducing them.
(Note: A bunch of tests were opted out of the new transform to preserve coverage. That landed in a previous commit to simplify revert cycles if they turn out to be needed.)
Diffstat (limited to 'llvm/lib/Transforms/Scalar/IndVarSimplify.cpp')
-rw-r--r-- | llvm/lib/Transforms/Scalar/IndVarSimplify.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp b/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp index f00b6a5ce3f..6737c62e94b 100644 --- a/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp +++ b/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp @@ -125,7 +125,7 @@ DisableLFTR("disable-lftr", cl::Hidden, cl::init(false), cl::desc("Disable Linear Function Test Replace optimization")); static cl::opt<bool> -LoopPredication("indvars-predicate-loops", cl::Hidden, cl::init(false), +LoopPredication("indvars-predicate-loops", cl::Hidden, cl::init(true), cl::desc("Predicate conditions in read only loops")); namespace { |