[PATCH] D56002: [AMDGPU] Fix a weird WWM intrinsic issue.
Neil Henning via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 7 04:52:34 PST 2019
sheredom added a comment.
@nhaehnle I've removed `canReadVGPR` as it only had the single callsite - but given @arsenm's comment I did not change `getOpRegClass`. I also don't feel super comfortable doing the change you suggested to `addUsersToMoveToVALUWorklist` as I'm worried that there will be non-LLVM-tested paths that I could trip up on with ease. I'd rather ship the commit as is if y'all are ok with it.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56002/new/
https://reviews.llvm.org/D56002
More information about the llvm-commits
mailing list