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

[Rahman Lavaee via Phabricator via llvm-commits]

rahmanl marked 6 inline comments as done.
rahmanl added a comment.

Thanks again @efriedma  and @tmsriram.



================
Comment at: llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp:1210-1213
+    if (MBB.isEndSection()) {
+      if (!MBB.sameSection(&MF->front())) {
+        if (MAI->hasDotTypeDotSizeDirective())
+          emitELFSizeDirective(CurrentSectionBeginSym);
----------------
Please excuse the nested //if// statement here. Our upcoming CFI and Exception patches will add one statement to each of the nested //if// blocks,


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

https://reviews.llvm.org/D76954