[PATCH] D99144: [NFC] Fix comment describing what EdgeBundles is

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 22 22:23:26 PDT 2021


mkazantsev created this revision.
mkazantsev added reviewers: RKSimon, dantrushin, skatkov, reames.
mkazantsev requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

The original comment says the same thing twice, and does not mention that
edges entering the block are also in the same bundle (which seems true from
what the underlying code is doing).


https://reviews.llvm.org/D99144

Files:
  llvm/include/llvm/CodeGen/EdgeBundles.h


Index: llvm/include/llvm/CodeGen/EdgeBundles.h
===================================================================
--- llvm/include/llvm/CodeGen/EdgeBundles.h
+++ llvm/include/llvm/CodeGen/EdgeBundles.h
@@ -8,7 +8,7 @@
 //
 // The EdgeBundles analysis forms equivalence classes of CFG edges such that all
 // edges leaving a machine basic block are in the same bundle, and all edges
-// leaving a basic block are in the same bundle.
+// entering a machine basic block are in the same bundle.
 //
 //===----------------------------------------------------------------------===//
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D99144.332527.patch
Type: text/x-patch
Size: 580 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210323/548b21b6/attachment.bin>


More information about the llvm-commits mailing list