diff options
author | Michael Kruse <llvm@meinersbur.de> | 2018-05-09 16:23:56 +0000 |
---|---|---|
committer | Michael Kruse <llvm@meinersbur.de> | 2018-05-09 16:23:56 +0000 |
commit | e330071b4354f8cdd8908a143d0f225a7ad0fd67 (patch) | |
tree | 3aa9596b03d356faffda128f253e9a8f652e5ccf /llvm/tools/llvm-rc/ResourceScriptParser.cpp | |
parent | cb7912cc0ff8e9adbfd1ffe4aa13310512ee737d (diff) | |
download | llvm-e330071b4354f8cdd8908a143d0f225a7ad0fd67.zip llvm-e330071b4354f8cdd8908a143d0f225a7ad0fd67.tar.gz llvm-e330071b4354f8cdd8908a143d0f225a7ad0fd67.tar.bz2 |
[ScopInfo] Remove bail out condition in buildMinMaxAccess().
The condition was introduced in r267142 to mitigate a long compile-time
case. In r306087, a max-computation limit was introduced that should
handle the same case while leaving the max disjuncts heuristic it
should have replaced intact.
Today, the max disjuncts bail-out causes problems in that it prematurely
stops SCoPs from being detected, e.g. in SPEC's lbm. This would hit less
like if isl_set_coalesce would be called after isl_set_remove_divs
(which makes more basic_set likely to be coalescable) instead of before.
This patch tries to remove the premature max-disjuncts bail-out
condition by using simple_hull() to reduce the computational overhead,
instead of directly invalidating that SCoP.
Differential Revision: https://reviews.llvm.org/D45066
Contributed-by: Sahil Girish Yerawar <cs15btech11044@iith.ac.in>
llvm-svn: 331891
Diffstat (limited to 'llvm/tools/llvm-rc/ResourceScriptParser.cpp')
0 files changed, 0 insertions, 0 deletions