================ @@ -81,10 +81,6 @@ class FunctionSummariesTy { I->second.MayInline = 0; } - void markReachedMaxBlockCount(const Decl *D) { - markShouldNotInline(D); - } ---------------- NagyDonat wrote:
I'm removing this function (by inlining its trivial definition) because I feel that its nondescript sneaky name significantly contributed to the fact that I did not know about this heuristic previously. **Please name functions clearly** – refer to their effects instead of just naming their call site (which is not helpful when I'm already reading the code that calls them)! https://github.com/llvm/llvm-project/pull/136720 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits