[PATCH] D129518: [BOLT] Add function layout class

[Maksim Panchenko via Phabricator via llvm-commits]

maksfb added inline comments.


================
Comment at: bolt/include/bolt/Core/FunctionLayout.h:11-14
+// new binary. Although the blocks of a function are contiguous, we can split
+// the layout into multiple fragments. The blocks within a fragment are
+// contiguous, but the fragments itself are disjoint. This is used to separate
+// the blocks into hot and cold sections.
----------------



================
Comment at: bolt/include/bolt/Core/FunctionLayout.h:61
+  FragmentNum Num;
+  const FunctionLayout *Layout;
+
----------------
Will the reference work?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129518