Kenneth Graunke <[email protected]> writes: > It is incorrect to assume that src[0] of a SEND-from-GRF opcode is the GRF. > VS_OPCODE_PULL_CONSTANT_LOAD_GEN7 uses an IMM as src[0], and stores the > GRF as src[1]. > > To be safe, loop over all the source registers and mark any GRFs. We > probably won't ever have more than one, but it's simpler to just check > all three rather than attempting to bail early. > > Fixes assertion failures in Unigine Sanctuary since we started making > register allocation rely on split_virtual_grfs working. (The register > classes were actually sufficient, we were just interpreting an IMM as > a virtual GRF number.) > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68637 > Signed-off-by: Kenneth Graunke <[email protected]> > Cc: [email protected]
These are: Reviewed-by: Eric Anholt <[email protected]>
pgpjjyMq4hDQB.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
