[PATCH] D87107: [AMDGPU] Target hook to apply target specific split constraint

Alexander via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 4 11:30:24 PDT 2020


alex-t added a comment.



> Also I still think that disabling a whole "endif" block is an overkill.

It only is disabled if S_OR_B64 exec, ... is in the middle of the block that should never happen.

  while (isBasicBlockPrologue(*J)) {
    if (IsExecRestore(&*J))
      return true;

assumes that if exec is restored in the block prologue it is valid


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87107/new/

https://reviews.llvm.org/D87107



More information about the llvm-commits mailing list