[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