[PATCH] D140540: [MachineTraceMetrics] Add local strategy

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineTraceMetrics.cpp:321-322
 
+// LocalEnsemble - Pick only the current basic block for the trace and do not
+// choose any predecessors/successors.
+class LocalEnsemble : public MachineTraceMetrics::Ensemble {
----------------
I see that this is replicating the previous code comment, but we shouldn't repeat the class/variable/function name in the block comment, and it should probably use "///" to produce the expected doxygen output.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140540