OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

intel/fs: shuffle_from_32bit_read at load_per_vertex_input at TCS/TES

Browse Source 
Previously, the shuffle function had a source/destination overlap that
needs to be avoided to use shuffle_from_32bit_read. As we can use for
the shuffle destination the destination of removed MOVs.

This change also avoids the internal MOVs done by the previous shuffle
to deal with possible overlaps.

Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
This commit is contained in:
Jose Maria Casanova Crespo
2018-06-09 11:46:07 +02:00
parent 5565630f85
commit 8003ae87f4
1 changed files with 8 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/intel/compiler/brw_fs_nir.cpp
View File

@@ -2665,13 +2665,10 @@ fs_visitor::nir_emit_tcs_intrinsic(const fs_builder &bld,
* or SSBOs. * or SSBOs.
*/ */
if (type_sz(dst.type) == 8) { if (type_sz(dst.type) == 8) {
shuffle_32bit_load_result_to_64bit_data( shuffle_from_32bit_read(bld,
bld, dst, retype(dst, BRW_REGISTER_TYPE_F), num_components); offset(orig_dst, bld, iter * 2),
retype(dst, BRW_REGISTER_TYPE_D),
for (unsigned c = 0; c < num_components; c++) { 0, num_components);
bld.MOV(offset(orig_dst, bld, iter * 2 + c),
offset(dst, bld, c));
}
} }
/* Copy the temporary to the destination to deal with writemasking. /* Copy the temporary to the destination to deal with writemasking.
@@ -3014,13 +3011,10 @@ fs_visitor::nir_emit_tes_intrinsic(const fs_builder &bld,
* or SSBOs. * or SSBOs.
*/ */
if (type_sz(dest.type) == 8) { if (type_sz(dest.type) == 8) {
shuffle_32bit_load_result_to_64bit_data( shuffle_from_32bit_read(bld,
bld, dest, retype(dest, BRW_REGISTER_TYPE_F), num_components); offset(orig_dest, bld, iter * 2),
retype(dest, BRW_REGISTER_TYPE_D),
for (unsigned c = 0; c < num_components; c++) { 0, num_components);
bld.MOV(offset(orig_dest, bld, iter * 2 + c),
offset(dest, bld, c));
}
} }
/* If we are loading double data and we need a second read message /* If we are loading double data and we need a second read message