Skip to content

Commit 007aa6d

Browse files
authored
[SLP] Increase UsesLimit to 64 (#99467)
Since commit 82b800e addressed the issue #99327 , we see some performance regression (13%) on some verilator generated C++ code. This is because the UsesLimit is set to 8, which is too small for the verilator generated code. I have analyzed the need for the UsesLimit from [1] and found that the UsesLimit should be at least 64 to cover most of these cases. Thus, This patch increases the UsesLimit to 64. Link: #99327 (comment) [1] Signed-off-by: Yangyu Chen <[email protected]>
1 parent 7d8375b commit 007aa6d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,7 @@ static const unsigned AliasedCheckLimit = 10;
209209

210210
// Limit of the number of uses for potentially transformed instructions/values,
211211
// used in checks to avoid compile-time explode.
212-
static constexpr int UsesLimit = 8;
212+
static constexpr int UsesLimit = 64;
213213

214214
// Another limit for the alias checks: The maximum distance between load/store
215215
// instructions where alias checks are done.

0 commit comments

Comments
 (0)