[PATCH] D127131: Pipe potentially invalid InstructionCost through CodeMetrics

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 9 12:37:38 PDT 2022


craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/include/llvm/Analysis/CodeMetrics.h:52
+  /// Code size cost of the analyzed blocks.
+  InstructionCost NumInsts = 0;
 
----------------
Is this field misnamed? It seems like that was already the case so I won't block the patch.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127131/new/

https://reviews.llvm.org/D127131



More information about the llvm-commits mailing list