[PATCH] D154102: Headers for basic blocks in control flow dot graphs

[Marek Sedláček via Phabricator via llvm-commits]

mark-sed created this revision.
mark-sed added reviewers: cdevadas, rcorcs, knaumov, sfertile.
Herald added subscribers: kerbowa, jvesely.
Herald added a project: All.
mark-sed requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

This is 3. out of 3 changes to the CFG pass that aims to make graphs more readable and contain more information.

This change adds separators for basic block names, which makes it easier to find a basic block based on its name and separates it from the code. Currently there is also a chance that the basic block label will be present twice, that is in case the basic block has explicit numbering, this change does not contain this bug.

If all 3 changes will be accepted the final graphs should look as in the attached image:
F28074492: new_cfg.png <https://reviews.llvm.org/F28074492>


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D154102

Files:
  llvm/include/llvm/Analysis/CFGPrinter.h
  llvm/test/Analysis/DotMachineCFG/AMDGPU/functions.mir
  llvm/test/Analysis/DotMachineCFG/AMDGPU/irreducible.mir
  llvm/test/Other/cfg_deopt_unreach.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D154102.535825.patch
Type: text/x-patch
Size: 9786 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230629/74ec1488/attachment.bin>