[PATCH] D118250: AMDGPU: Mark control flow intrinsics non-duplicable
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 26 07:25:57 PST 2022
arsenm requested changes to this revision.
arsenm added a comment.
This revision now requires changes to proceed.
I don't think this is a good idea. We don't actually need a structured CFG at this point, and tail duplicating isn't exactly unstructuring anyway. This is not an alternative to fixing the LiveVariables update problem, it's just the testcase that broke happened to have appeared due to tail duplication
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D118250/new/
https://reviews.llvm.org/D118250
More information about the llvm-commits
mailing list