diff options
author | Max Kazantsev <mkazantsev@azul.com> | 2023-03-22 14:29:12 +0700 |
---|---|---|
committer | Max Kazantsev <mkazantsev@azul.com> | 2023-03-22 15:15:26 +0700 |
commit | 7b83a1438f9af064851bac3d74e05f794088d6d6 (patch) | |
tree | 545eab4692a824d077307e1db578833c1ebfdaea /flang/lib/Frontend/CompilerInvocation.cpp | |
parent | f809eb4db2d14a5a529f9f440b849b7489292976 (diff) | |
download | llvm-7b83a1438f9af064851bac3d74e05f794088d6d6.zip llvm-7b83a1438f9af064851bac3d74e05f794088d6d6.tar.gz llvm-7b83a1438f9af064851bac3d74e05f794088d6d6.tar.bz2 |
[GuardWidening] Improve analysis of potential widening into hotter block, try 2
The initial version was reverted because it looped infinitely if the likely successor
isn't properly dominated by the predecessor. In practice it means that we went up the
CFG through backedge and looped infinitely.
I also added some paranoid assertion checks to make sure that every other invariant
holds. I also found a hypothetical situation when we may go past the dominated block
while following the likely successors (it means that in fact the dominated block is
dynamically not reachable from dominating block) and explicitly prohibited this, though
I don't have a motivating test showing that it's a real problem.
https://reviews.llvm.org/D146276
Diffstat (limited to 'flang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions