[PATCH] D50982: [AMDGPU] Legalize VGPR Rsrc operands for MUBUF instructions

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 23 00:59:30 PDT 2018


nhaehnle added a comment.

I find it a bit concerning that we seem to have different semantics for these instructions depending on the environment.

Graphics really wants to legalize the instructions using a single v_readfirstlane (without the loop overhead), because that's what the relevant semantics are there. This patch doesn't change that, so it's not too bad, but please keep it in mind.


https://reviews.llvm.org/D50982





More information about the llvm-commits mailing list