[Mesa-dev] [PATCH 4/4] llvmpipe: rename polygon offset fields to something more specific

Jose Fonseca jfonseca at vmware.com
Thu Feb 21 13:49:42 PST 2013


Series is
Reviewed-by: Jose Fonseca <jfonseca at vmware.com>


----- Original Message -----
> ---
>  src/gallium/drivers/llvmpipe/lp_state_setup.c |   15 +++++++++------
>  src/gallium/drivers/llvmpipe/lp_state_setup.h |    4 ++--
>  2 files changed, 11 insertions(+), 8 deletions(-)
> 
> diff --git a/src/gallium/drivers/llvmpipe/lp_state_setup.c
> b/src/gallium/drivers/llvmpipe/lp_state_setup.c
> index f44eed4..edd5d0f 100644
> --- a/src/gallium/drivers/llvmpipe/lp_state_setup.c
> +++ b/src/gallium/drivers/llvmpipe/lp_state_setup.c
> @@ -288,12 +288,15 @@ lp_do_offset_tri(struct gallivm_state *gallivm,
>     dzdx = LLVMBuildExtractElement(b, dzdxdzdy, zeroi, "");
>     dzdy = LLVMBuildExtractElement(b, dzdxdzdy, onei, "");
>  
> -   /* zoffset = offset->units + MAX2(dzdx, dzdy) * offset->scale */
> +   /* zoffset = pgon_offset_units + MAX2(dzdx, dzdy) * pgon_offset_scale */
>     max = LLVMBuildFCmp(b, LLVMRealUGT, dzdx, dzdy, "");
>     max_value = LLVMBuildSelect(b, max, dzdx, dzdy, "max");
>  
> -   mult = LLVMBuildFMul(b, max_value, lp_build_const_float(gallivm,
> key->scale), "");
> -   zoffset = LLVMBuildFAdd(b, lp_build_const_float(gallivm, key->units),
> mult, "zoffset");
> +   mult = LLVMBuildFMul(b, max_value,
> +                        lp_build_const_float(gallivm,
> key->pgon_offset_scale), "");
> +   zoffset = LLVMBuildFAdd(b,
> +                           lp_build_const_float(gallivm,
> key->pgon_offset_units),
> +                           mult, "zoffset");
>  
>     /* yuck */
>     shuffles[0] = twoi;
> @@ -639,7 +642,7 @@ init_args(struct gallivm_state *gallivm,
>     ooa = vec4f_from_scalar(gallivm, ooa, "");
>  
>     /* tri offset calc shares a lot of arithmetic, do it here */
> -   if (key->scale != 0.0f || key->units != 0.0f) {
> +   if (key->pgon_offset_scale != 0.0f || key->pgon_offset_units != 0.0f) {
>        lp_do_offset_tri(gallivm, args, key, ooa, dxy01, dxy20, attr_pos);
>     }
>  
> @@ -818,8 +821,8 @@ lp_make_setup_variant_key(struct llvmpipe_context *lp,
>     assert(key->spec_slot   == lp->color_slot [1]);
>     assert(key->bspec_slot  == lp->bcolor_slot[1]);
>  
> -   key->units = (float) (lp->rasterizer->offset_units * lp->mrd);
> -   key->scale = lp->rasterizer->offset_scale;
> +   key->pgon_offset_units = (float) (lp->rasterizer->offset_units *
> lp->mrd);
> +   key->pgon_offset_scale = lp->rasterizer->offset_scale;
>     key->pad = 0;
>     memcpy(key->inputs, fs->inputs, key->num_inputs * sizeof key->inputs[0]);
>     for (i = 0; i < key->num_inputs; i++) {
> diff --git a/src/gallium/drivers/llvmpipe/lp_state_setup.h
> b/src/gallium/drivers/llvmpipe/lp_state_setup.h
> index e0abe46..73d40a5 100644
> --- a/src/gallium/drivers/llvmpipe/lp_state_setup.h
> +++ b/src/gallium/drivers/llvmpipe/lp_state_setup.h
> @@ -27,8 +27,8 @@ struct lp_setup_variant_key {
>     unsigned twoside:1;
>     unsigned pad:5;
>  
> -   float units;
> -   float scale;
> +   float pgon_offset_units;
> +   float pgon_offset_scale;
>     struct lp_shader_input inputs[PIPE_MAX_SHADER_INPUTS];
>  };
>  
> --
> 1.7.3.4
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 


More information about the mesa-dev mailing list