[llvm] [BOLT] Remove mutable from BB::LayoutIndex (PR #93224)
Amir Ayupov via llvm-commits
llvm-commits at lists.llvm.org
Thu May 30 10:47:23 PDT 2024
https://github.com/aaupov requested changes to this pull request.
Thank you for working on this. The diff is moving in the right direction, but as evidenced by the amount of code to avoid setting BB->LayoutIndex and associated overheads, it's not a good idea and we should still set it.
Please define a helper that given a block order sets BB->LayoutIndex, as part of FunctionLayout class.
https://github.com/llvm/llvm-project/pull/93224
More information about the llvm-commits
mailing list