[PATCH] D133067: [AMDGPU] W/a hazard if 64 bit shift amount is a highest allocated VGPR

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 02:20:45 PDT 2022


foad added a comment.

> Also we could still constrain src0 of these to a decimated RC in the finalizeLowering and keep this w/a just in case further optimizations would replace the operand.

If you use a different RC for src0 then surely you don't need the workaround? The rest of the compiler should respect the RC, and should not "replace the operand" with a physreg that is not in the right RC.


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

https://reviews.llvm.org/D133067



More information about the llvm-commits mailing list