[PATCH] D33306: [IfConversion] Make the ifcvt-limit command line option work at the function level and remove compares with global Statistic variables

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added a comment.

So, I think this was actually working as intended.

The limit is off by default. Turning the limit on I think is essentially used to bisect to a particular if conversion within a given run over a module for debugging.

I'm not suggesting it should keep using statistics to do this bisection, just pointing out that the "global limit" thing is actually "global stop counter for debugging and bisection". Renaming it would seem appropriate. =]


https://reviews.llvm.org/D33306