[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