[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