[PATCH] D98631: AMDGPU: Don't require registers for s_setpc_b64 operand

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 15 10:04:07 PDT 2021


rampitec requested changes to this revision.
rampitec added a comment.
This revision now requires changes to proceed.

In D98631#2626123 <https://reviews.llvm.org/D98631#2626123>, @dp wrote:

>> I don't see any restriction noted that this must be a register, so just allow any SSrc_b64 like normal.
>
> Documentation is not clear here, however sp3 does not allow constants for src0 (as opposed to null).

+1. It is defined as OPR_SREG.


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

https://reviews.llvm.org/D98631



More information about the llvm-commits mailing list