[PATCH] D118975: [AMDGPU] Allow hoisting of some VALU compare instructions

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 7 11:34:24 PST 2022


nhaehnle added a comment.

I suspect that V_CNDMASK uses are fairly common, and those implicitly mask with EXEC as well.

This also doesn't address more complex cases like

  %0 = V_CMP ...
  %1 = V_CMP ...
  %2 = S_OR %0, %1
  V_CNDMASK lhs, rhs, %2

though perhaps the change is good as-is for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118975



More information about the llvm-commits mailing list