[PATCH] D63279: [Analyzer] Unroll for-loops where the upper boundary is a variable with know value
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 14 15:30:18 PDT 2019
NoQ added a comment.
A quick archaeological dig suggests that i had no concerns with turning on unrolling by default back in 2017: http://lists.llvm.org/pipermail/cfe-dev/2017-August/055221.html - and i don't think i had any new concerns since then. I guess it'll be beneficial to make one more large-codebase run to make sure it still works, but other than that the only reason it's not enabled by default yet is because we forgot to do that :( This patch is good but not a must-have to turn loop unrolling on by default.
CHANGES SINCE LAST ACTION
More information about the cfe-commits