summaryrefslogtreecommitdiffstats
path: root/src/compiler
diff options
context:
space:
mode:
authorFrancisco Jerez <currojerez@riseup.net>2016-11-01 11:56:13 -0700
committerEmil Velikov <emil.l.velikov@gmail.com>2016-11-09 14:00:31 +0000
commit2789bfdbb59c4528a248dd8c02124dd384caefe6 (patch)
tree5fd379a8bf46bd76b3433c8ef403cdc52070dc8f /src/compiler
parentda1ac6bc4642f0c2ab10f557d7749fc6d41dc4b7 (diff)
downloadexternal_mesa3d-2789bfdbb59c4528a248dd8c02124dd384caefe6.zip
external_mesa3d-2789bfdbb59c4528a248dd8c02124dd384caefe6.tar.gz
external_mesa3d-2789bfdbb59c4528a248dd8c02124dd384caefe6.tar.bz2
nir: Flip gl_SamplePosition in nir_lower_wpos_ytransform().
Assuming the hardware is set up to use a screen coordinate system flipped vertically with respect to the GL's window coordinate system, the SYSTEM_VALUE_SAMPLE_POS vector will also be flipped vertically with respect to the value expected by the GL, so we need to give it the same treatment as gl_FragCoord. Fixes the following CTS tests on i965: ES31-CTS.functional.shaders.multisample_interpolation.interpolate_at_offset.at_sample_position.default_framebuffer ES31-CTS.functional.shaders.sample_variables.sample_pos.correctness.default_framebuffer when run with any multisample configuration, e.g. rgba8888d24s8ms4. Cc: <mesa-stable@lists.freedesktop.org> Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com> (cherry picked from commit f3d387867f74ae758b41168f23992671f7dce254)
Diffstat (limited to 'src/compiler')
-rw-r--r--src/compiler/nir/nir_lower_wpos_ytransform.c24
1 files changed, 24 insertions, 0 deletions
diff --git a/src/compiler/nir/nir_lower_wpos_ytransform.c b/src/compiler/nir/nir_lower_wpos_ytransform.c
index 173f058..f211c73 100644
--- a/src/compiler/nir/nir_lower_wpos_ytransform.c
+++ b/src/compiler/nir/nir_lower_wpos_ytransform.c
@@ -273,6 +273,26 @@ lower_interp_var_at_offset(lower_wpos_ytransform_state *state,
}
static void
+lower_load_sample_pos(lower_wpos_ytransform_state *state,
+ nir_intrinsic_instr *intr)
+{
+ nir_builder *b = &state->b;
+ b->cursor = nir_after_instr(&intr->instr);
+
+ nir_ssa_def *pos = &intr->dest.ssa;
+ nir_ssa_def *scale = nir_channel(b, get_transform(state), 0);
+ nir_ssa_def *neg_scale = nir_channel(b, get_transform(state), 2);
+ /* Either y or 1-y for scale equal to 1 or -1 respectively. */
+ nir_ssa_def *flipped_y =
+ nir_fadd(b, nir_fmax(b, neg_scale, nir_imm_float(b, 0.0)),
+ nir_fmul(b, nir_channel(b, pos, 1), scale));
+ nir_ssa_def *flipped_pos = nir_vec2(b, nir_channel(b, pos, 0), flipped_y);
+
+ nir_ssa_def_rewrite_uses_after(&intr->dest.ssa, nir_src_for_ssa(flipped_pos),
+ flipped_pos->parent_instr);
+}
+
+static void
lower_wpos_ytransform_block(lower_wpos_ytransform_state *state, nir_block *block)
{
nir_foreach_instr_safe(instr, block) {
@@ -287,6 +307,10 @@ lower_wpos_ytransform_block(lower_wpos_ytransform_state *state, nir_block *block
/* gl_FragCoord should not have array/struct deref's: */
assert(dvar->deref.child == NULL);
lower_fragcoord(state, intr);
+ } else if (var->data.mode == nir_var_system_value &&
+ var->data.location == SYSTEM_VALUE_SAMPLE_POS) {
+ assert(dvar->deref.child == NULL);
+ lower_load_sample_pos(state, intr);
}
} else if (intr->intrinsic == nir_intrinsic_interp_var_at_offset) {
lower_interp_var_at_offset(state, intr);