[PATCH] D76954: LLVM support for BB-cluster sections

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 10 16:09:56 PDT 2020


efriedma accepted this revision.
efriedma added a comment.

LGTM with one minor suggestion



================
Comment at: llvm/lib/CodeGen/MIRPrinter.cpp:646
   }
-  if (MBB.getSectionType() != MBBS_None) {
+  if (MBB.getParent()->hasBBSections()) {
     OS << (HasAttributes ? ", " : " (");
----------------
Maybe just print all non-zero SectionIDs, instead of checking hasBBSections()?


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

https://reviews.llvm.org/D76954





More information about the llvm-commits mailing list