[PATCH] D123807: llvm-reduce: Clone properties of blocks

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 14 11:05:12 PDT 2022


arsenm added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/MachineBasicBlock.h:1089-1090
 
+  /// Return probability of the edge from this block to MBB. This method should
+  /// NOT be called directly, but by using getEdgeProbability method from
+  /// MachineBranchProbabilityInfo class.
----------------
aeubanks wrote:
> this should be updated, it contradicts the change description
I'm not sure if the comment should be removed or not. I'm removing the language enforcement mechanism. Not sure if the original justification for this comment makes sense or not


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

https://reviews.llvm.org/D123807



More information about the llvm-commits mailing list