[PATCH] D107128: Explain the symbols of basic block clusters with an example in the header comments.

Snehasish Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 29 19:26:17 PDT 2021


snehasish accepted this revision.
snehasish added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/CodeGen/BasicBlockSections.cpp:37
+//   `foo.__part.1` will reference the beginning of this section.
+// * The only cold basic block 4 is placed in one section, and a new symbol
+//   `foo.cold` will point to it.
----------------
How about a minor edit here:

... basic block 4 (note that it is not referenced in the list) is placed in one section ...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107128



More information about the llvm-commits mailing list