diff options
-rw-r--r-- | src/gallium/drivers/freedreno/a3xx/fd3_draw.c | 27 | ||||
-rw-r--r-- | src/gallium/drivers/freedreno/a4xx/fd4_draw.c | 32 | ||||
-rw-r--r-- | src/gallium/drivers/freedreno/a5xx/fd5_draw.c | 32 | ||||
-rw-r--r-- | src/gallium/drivers/freedreno/ir3/ir3_shader.h | 51 |
4 files changed, 63 insertions, 79 deletions
diff --git a/src/gallium/drivers/freedreno/a3xx/fd3_draw.c b/src/gallium/drivers/freedreno/a3xx/fd3_draw.c index b3cd4619559..afa2c3c7ef7 100644 --- a/src/gallium/drivers/freedreno/a3xx/fd3_draw.c +++ b/src/gallium/drivers/freedreno/a3xx/fd3_draw.c @@ -100,32 +100,13 @@ fixup_shader_state(struct fd_context *ctx, struct ir3_shader_key *key) struct ir3_shader_key *last_key = &fd3_ctx->last_key; if (!ir3_shader_key_equal(last_key, key)) { - if (last_key->has_per_samp || key->has_per_samp) { - if ((last_key->vsaturate_s != key->vsaturate_s) || - (last_key->vsaturate_t != key->vsaturate_t) || - (last_key->vsaturate_r != key->vsaturate_r)) - ctx->dirty |= FD_SHADER_DIRTY_VP; - - if ((last_key->fsaturate_s != key->fsaturate_s) || - (last_key->fsaturate_t != key->fsaturate_t) || - (last_key->fsaturate_r != key->fsaturate_r)) - ctx->dirty |= FD_SHADER_DIRTY_FP; + if (ir3_shader_key_changes_fs(last_key, key)) { + ctx->dirty |= FD_SHADER_DIRTY_FP; } - if (last_key->vclamp_color != key->vclamp_color) + if (ir3_shader_key_changes_vs(last_key, key)) { ctx->dirty |= FD_SHADER_DIRTY_VP; - - if (last_key->fclamp_color != key->fclamp_color) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->color_two_side != key->color_two_side) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->half_precision != key->half_precision) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->ucp_enables != key->ucp_enables) - ctx->dirty |= FD_SHADER_DIRTY_FP | FD_SHADER_DIRTY_VP; + } fd3_ctx->last_key = *key; } diff --git a/src/gallium/drivers/freedreno/a4xx/fd4_draw.c b/src/gallium/drivers/freedreno/a4xx/fd4_draw.c index 641c58a15bd..c0ada87f83e 100644 --- a/src/gallium/drivers/freedreno/a4xx/fd4_draw.c +++ b/src/gallium/drivers/freedreno/a4xx/fd4_draw.c @@ -84,37 +84,13 @@ fixup_shader_state(struct fd_context *ctx, struct ir3_shader_key *key) struct ir3_shader_key *last_key = &fd4_ctx->last_key; if (!ir3_shader_key_equal(last_key, key)) { - if (last_key->has_per_samp || key->has_per_samp) { - if ((last_key->vsaturate_s != key->vsaturate_s) || - (last_key->vsaturate_t != key->vsaturate_t) || - (last_key->vsaturate_r != key->vsaturate_r) || - (last_key->vastc_srgb != key->vastc_srgb)) - ctx->dirty |= FD_SHADER_DIRTY_VP; - - if ((last_key->fsaturate_s != key->fsaturate_s) || - (last_key->fsaturate_t != key->fsaturate_t) || - (last_key->fsaturate_r != key->fsaturate_r) || - (last_key->fastc_srgb != key->fastc_srgb)) - ctx->dirty |= FD_SHADER_DIRTY_FP; + if (ir3_shader_key_changes_fs(last_key, key)) { + ctx->dirty |= FD_SHADER_DIRTY_FP; } - if (last_key->vclamp_color != key->vclamp_color) + if (ir3_shader_key_changes_vs(last_key, key)) { ctx->dirty |= FD_SHADER_DIRTY_VP; - - if (last_key->fclamp_color != key->fclamp_color) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->color_two_side != key->color_two_side) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->half_precision != key->half_precision) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->rasterflat != key->rasterflat) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->ucp_enables != key->ucp_enables) - ctx->dirty |= FD_SHADER_DIRTY_FP | FD_SHADER_DIRTY_VP; + } fd4_ctx->last_key = *key; } diff --git a/src/gallium/drivers/freedreno/a5xx/fd5_draw.c b/src/gallium/drivers/freedreno/a5xx/fd5_draw.c index d9fce2f7ee8..b7417ec92a9 100644 --- a/src/gallium/drivers/freedreno/a5xx/fd5_draw.c +++ b/src/gallium/drivers/freedreno/a5xx/fd5_draw.c @@ -77,37 +77,13 @@ fixup_shader_state(struct fd_context *ctx, struct ir3_shader_key *key) struct ir3_shader_key *last_key = &fd5_ctx->last_key; if (!ir3_shader_key_equal(last_key, key)) { - if (last_key->has_per_samp || key->has_per_samp) { - if ((last_key->vsaturate_s != key->vsaturate_s) || - (last_key->vsaturate_t != key->vsaturate_t) || - (last_key->vsaturate_r != key->vsaturate_r) || - (last_key->vastc_srgb != key->vastc_srgb)) - ctx->dirty |= FD_SHADER_DIRTY_VP; - - if ((last_key->fsaturate_s != key->fsaturate_s) || - (last_key->fsaturate_t != key->fsaturate_t) || - (last_key->fsaturate_r != key->fsaturate_r) || - (last_key->fastc_srgb != key->fastc_srgb)) - ctx->dirty |= FD_SHADER_DIRTY_FP; + if (ir3_shader_key_changes_fs(last_key, key)) { + ctx->dirty |= FD_SHADER_DIRTY_FP; } - if (last_key->vclamp_color != key->vclamp_color) + if (ir3_shader_key_changes_vs(last_key, key)) { ctx->dirty |= FD_SHADER_DIRTY_VP; - - if (last_key->fclamp_color != key->fclamp_color) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->color_two_side != key->color_two_side) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->half_precision != key->half_precision) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->rasterflat != key->rasterflat) - ctx->dirty |= FD_SHADER_DIRTY_FP; - - if (last_key->ucp_enables != key->ucp_enables) - ctx->dirty |= FD_SHADER_DIRTY_FP | FD_SHADER_DIRTY_VP; + } fd5_ctx->last_key = *key; } diff --git a/src/gallium/drivers/freedreno/ir3/ir3_shader.h b/src/gallium/drivers/freedreno/ir3/ir3_shader.h index 052a563b945..59584ec5b7b 100644 --- a/src/gallium/drivers/freedreno/ir3/ir3_shader.h +++ b/src/gallium/drivers/freedreno/ir3/ir3_shader.h @@ -105,6 +105,57 @@ ir3_shader_key_equal(struct ir3_shader_key *a, struct ir3_shader_key *b) return a->global == b->global; } +/* will the two keys produce different lowering for a fragment shader? */ +static inline bool +ir3_shader_key_changes_fs(struct ir3_shader_key *key, struct ir3_shader_key *last_key) +{ + if (last_key->has_per_samp || key->has_per_samp) { + if ((last_key->fsaturate_s != key->fsaturate_s) || + (last_key->fsaturate_t != key->fsaturate_t) || + (last_key->fsaturate_r != key->fsaturate_r) || + (last_key->fastc_srgb != key->fastc_srgb)) + return true; + } + + if (last_key->fclamp_color != key->fclamp_color) + return true; + + if (last_key->color_two_side != key->color_two_side) + return true; + + if (last_key->half_precision != key->half_precision) + return true; + + if (last_key->rasterflat != key->rasterflat) + return true; + + if (last_key->ucp_enables != key->ucp_enables) + return true; + + return false; +} + +/* will the two keys produce different lowering for a vertex shader? */ +static inline bool +ir3_shader_key_changes_vs(struct ir3_shader_key *key, struct ir3_shader_key *last_key) +{ + if (last_key->has_per_samp || key->has_per_samp) { + if ((last_key->vsaturate_s != key->vsaturate_s) || + (last_key->vsaturate_t != key->vsaturate_t) || + (last_key->vsaturate_r != key->vsaturate_r) || + (last_key->vastc_srgb != key->vastc_srgb)) + return true; + } + + if (last_key->vclamp_color != key->vclamp_color) + return true; + + if (last_key->ucp_enables != key->ucp_enables) + return true; + + return false; +} + struct ir3_shader_variant { struct fd_bo *bo; |