[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