> 
> On Apr 21, 2016, at 11:20 AM, Ilia Mirkin <imir...@alum.mit.edu> wrote:
> 
> On Thu, Apr 21, 2016 at 12:17 PM, Tim Rowley <timothy.o.row...@intel.com> 
> wrote:
>> ---
>> src/gallium/drivers/swr/swr_query.cpp | 2 +-
>> src/gallium/drivers/swr/swr_shader.h  | 8 ++++----
>> 2 files changed, 5 insertions(+), 5 deletions(-)
>> 
>> diff --git a/src/gallium/drivers/swr/swr_query.cpp 
>> b/src/gallium/drivers/swr/swr_query.cpp
>> index e4b8b68..92a1a0b 100644
>> --- a/src/gallium/drivers/swr/swr_query.cpp
>> +++ b/src/gallium/drivers/swr/swr_query.cpp
>> @@ -314,7 +314,7 @@ swr_check_render_cond(struct pipe_context *pipe)
>>    b = pipe->get_query_result(
>>       pipe, ctx->render_cond_query, wait, (union pipe_query_result 
>> *)&result);
>>    if (b)
>> -      return (!result == ctx->render_cond_cond);
>> +      return (!(result == ctx->render_cond_cond));
> 
> Perhaps simpler expressed as
> 
> result != ctx->render_cond_cond
> 
> ?
> 
This cam from the llvmpipe code that I used as the original template and was 
fixed by Roland
on 25-MAY-15 in commit 6a111e54d7578abee6bce4a75ce1399ed369ab5f.

Correct fix should be:
        return ((!result) == lp->render_cond_cond);


>>    else
>>       return TRUE;
>> }
>> diff --git a/src/gallium/drivers/swr/swr_shader.h 
>> b/src/gallium/drivers/swr/swr_shader.h
>> index 3f79570..11d50c3 100644
>> --- a/src/gallium/drivers/swr/swr_shader.h
>> +++ b/src/gallium/drivers/swr/swr_shader.h
>> @@ -23,10 +23,10 @@
>> 
>> #pragma once
>> 
>> -class swr_vertex_shader;
>> -class swr_fragment_shader;
>> -class swr_jit_fs_key;
>> -class swr_jit_vs_key;
>> +struct swr_vertex_shader;
>> +struct swr_fragment_shader;
>> +struct swr_jit_fs_key;
>> +struct swr_jit_vs_key;
>> 
>> PFN_VERTEX_FUNC
>> swr_compile_vs(struct swr_context *ctx, swr_jit_vs_key &key);
>> --
>> 1.9.1
>> 
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to