[PATCH] D73142: Heat Coloring for CFGPrinter and CallPrinter

Kirill Naumov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 28 13:42:10 PST 2020


knaumov updated this revision to Diff 247362.
knaumov marked 2 inline comments as done.
knaumov added a comment.

Fixes:

- Restoring the effect of clang-format

The effect of clang-format was restored for better readability of the patch

- Deleted Heuristic fully

As I said in the previous diffs, Heuristic bool was always true and never changing.

- Transferred CFGPrinter back to Function Pass

The patch was made module in order to implement the comparison between BB across functions. As this patch is not targeting this behavior, the pass was decided to remain Function Pass

- Small stylistics changes


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73142/new/

https://reviews.llvm.org/D73142

Files:
  llvm/include/llvm/Analysis/CFGPrinter.h
  llvm/include/llvm/Analysis/HeatUtils.h
  llvm/lib/Analysis/CFGPrinter.cpp
  llvm/lib/Analysis/CMakeLists.txt
  llvm/lib/Analysis/CallPrinter.cpp
  llvm/lib/Analysis/DomPrinter.cpp
  llvm/lib/Analysis/HeatUtils.cpp
  llvm/lib/Analysis/RegionPrinter.cpp
  llvm/lib/Transforms/Scalar/NewGVN.cpp
  llvm/test/Other/2007-06-05-PassID.ll
  llvm/test/Other/cfg-printer-branch-weights-percent.ll
  llvm/test/Other/cfg-printer-branch-weights.ll
  polly/lib/Analysis/ScopGraphPrinter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D73142.247362.patch
Type: text/x-patch
Size: 37589 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200228/38dd62e6/attachment.bin>


More information about the llvm-commits mailing list