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

[Thorsten via Phabricator via llvm-commits]

tschuett added a comment.

Just as a comment. Maybe down the line, you could replace the `unsigned` with a `FragmentId`, a wrapper around an `unsigned`,  to avoid confusion.

I have a feeling that you want to deprecate `FunctionLayout::blocks()`. There is probably a `LLVM_DEPRECATED` macro.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129518