[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