[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