[llvm] [LLVM] Successor count added to InstCount (PR #171670)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 21 09:57:58 PST 2025


================
@@ -412,9 +413,12 @@ void PassBuilder::invokePipelineEarlySimplificationEPCallbacks(
 // Helper to add AnnotationRemarksPass.
 static void addAnnotationRemarksPass(ModulePassManager &MPM) {
   MPM.addPass(createModuleToFunctionPassAdaptor(AnnotationRemarksPass()));
-  // Count the types of instructions used
-  if (AreStatisticsEnabled())
+  // Count the types of instructions used in a module if statistics are enabled.
----------------
boomanaiden154 wrote:

This comment should be updated to cover the function properties case. The addition of "in a module" is kind of redundant.

https://github.com/llvm/llvm-project/pull/171670


More information about the llvm-commits mailing list