[PATCH] D117814: [AMDGPU] Do not ignore exec use where exec is read as data

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 21 03:38:43 PST 2022


foad reopened this revision.
foad added a comment.
This revision is now accepted and ready to land.

> Compares, v_cndmask_b32, and v_readfirstlane_b32 use EXEC in a way which modifies the result.

This doesn't make any sense to me for cndmask. It handles EXEC exactly the same as any other VALU instruction: it writes a result in any lane which is enabled by EXEC, and does nothing for any lane that is disabled.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117814



More information about the llvm-commits mailing list