[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