[PATCH] D36553: [InstCombine] Add a DEBUG_COUNTER to InstCombine to limit how many instructions are visited for debug

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 9 23:50:55 PDT 2017


davide added a comment.

> I also wonder if we should change the DEBUG_COUNTER macro to have the semicolon outside like we do for STATISTIC. Since they are often going to appear at the top of a file near STATISTIC the inconsistency seems likely to cause people to add a semicolon after DEBUG_COUNTER anyway.

I think it boils down to a matter of tastes, and it's probably better for consistency.


https://reviews.llvm.org/D36553





More information about the llvm-commits mailing list