[PATCH] D131150: [AMDGPU][SIAnnotateCF] Support Conditions Using a Common Basic Block
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 4 07:12:30 PDT 2022
arsenm added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/SIAnnotateControlFlow.cpp:354
+ // then/else block, we'd have pushed that block twice on the stack.
+ while (isTopOfStack(BB))
Changed |= closeControlFlow(BB);
----------------
arsenm wrote:
> Could you construct a situation where the block appears more times, and not consecutively? It may be better to change the stack to a SetVector
The answer is probably no since this is supposed to be structurized already
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D131150/new/
https://reviews.llvm.org/D131150
More information about the llvm-commits
mailing list