[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