[PATCH] D137467: [NFC][BlockPlacement]Add an option to renumber blocks based on function layout order.
Mingming Liu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 7 07:51:20 PST 2022
mingmingl added a comment.
In D137467#3911238 <https://reviews.llvm.org/D137467#3911238>, @davidxl wrote:
> I see. The DOT CFG graph dump uses the layout order instead of the MBB number. Another way to fix this is to use the following in the node label: BBName[MBBNum][layout-order], but making the MBB num and layout number consistent under an option is fine to me.
Using 'BBName[MBBNum][layout-order]' would keep more information; but it requires more implementation work.
Going to submit this. Thanks for reviews!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137467/new/
https://reviews.llvm.org/D137467
More information about the llvm-commits
mailing list