diff options
| author | Jason Ekstrand <jason.ekstrand@intel.com> | 2018-01-05 18:50:14 -0800 |
|---|---|---|
| committer | Jason Ekstrand <jason.ekstrand@intel.com> | 2018-01-11 14:31:47 -0800 |
| commit | c3d802d68e222bf4030ab4e487501dfbcb34608d (patch) | |
| tree | 53c31005f645a0b7d32197013bc9860dc9d5236d | |
| parent | 3d2b157e23c9d66df97d59be6efd1098878cc110 (diff) | |
i965: Use UD types for gl_SampleID setup
We already had to switch all of the W types to UW to prevent issues
with vector immediates on gen10. We may as well use unsigned types
everywhere.
Reviewed-by: Matt Turner <mattst88@gmail.com>
| -rw-r--r-- | src/intel/compiler/brw_fs.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp index 09bfc9c552..faf3a52c26 100644 --- a/src/intel/compiler/brw_fs.cpp +++ b/src/intel/compiler/brw_fs.cpp @@ -1219,7 +1219,7 @@ fs_visitor::emit_sampleid_setup() assert(devinfo->gen >= 6); const fs_builder abld = bld.annotate("compute sample id"); - fs_reg *reg = new(this->mem_ctx) fs_reg(vgrf(glsl_type::int_type)); + fs_reg *reg = new(this->mem_ctx) fs_reg(vgrf(glsl_type::uint_type)); if (!key->multisample_fbo) { /* As per GL_ARB_sample_shading specification: @@ -1264,7 +1264,7 @@ fs_visitor::emit_sampleid_setup() abld.AND(*reg, tmp, brw_imm_w(0xf)); } else { const fs_reg t1 = component(fs_reg(VGRF, alloc.allocate(1), - BRW_REGISTER_TYPE_D), 0); + BRW_REGISTER_TYPE_UD), 0); const fs_reg t2(VGRF, alloc.allocate(1), BRW_REGISTER_TYPE_UW); /* The PS will be run in MSDISPMODE_PERSAMPLE. For example with @@ -1291,7 +1291,7 @@ fs_visitor::emit_sampleid_setup() * accomodate 16x MSAA. */ abld.exec_all().group(1, 0) - .AND(t1, fs_reg(retype(brw_vec1_grf(0, 0), BRW_REGISTER_TYPE_D)), + .AND(t1, fs_reg(retype(brw_vec1_grf(0, 0), BRW_REGISTER_TYPE_UD)), brw_imm_ud(0xc0)); abld.exec_all().group(1, 0).SHR(t1, t1, brw_imm_d(5)); |