[PATCH] D137615: [BOLT][Instrumentation] Instrument leaves from spanning tree

Alexey Moksyakov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 12 03:29:26 PST 2023


yavtuk added a comment.

@rafauler you’re right, thanks for explanation.
I was confused by the following checking

if (STOutSet[&BB].size() == 0)

Because if for some reason the BB is not in the spanning tree, the default constructor for this base block is called and it is inserted into the STOutSet with 0 number of successros.
Therefore the insturmentation snipet is inserted.
But now there is only one reason for that, a BB is a tree leaf.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137615



More information about the llvm-commits mailing list