On 11 August 2017 at 12:31, Tapani Pälli <tapani.pa...@intel.com> wrote:
> On 08/11/2017 02:23 PM, Grazvydas Ignotas wrote:
>>
>> On Fri, Aug 11, 2017 at 8:52 AM, Kenneth Graunke <kenn...@whitecape.org>
>> wrote:
>>>
>>> This should hopefully fix build issues on 32-bit Android-x86.
>>>
>>> Cc: Mauro Rossi <issor.or...@gmail.com>
>>> Cc: Tapani Pälli <tapani.pa...@intel.com>
>>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102050
>>> ---
>>>   src/mesa/drivers/dri/i965/intel_buffer_objects.c | 2 ++
>>>   1 file changed, 2 insertions(+)
>>>
>>> Mauro, hopefully this helps?
>>>
>>> diff --git a/src/mesa/drivers/dri/i965/intel_buffer_objects.c
>>> b/src/mesa/drivers/dri/i965/intel_buffer_objects.c
>>> index ee591168283..9afd98edb87 100644
>>> --- a/src/mesa/drivers/dri/i965/intel_buffer_objects.c
>>> +++ b/src/mesa/drivers/dri/i965/intel_buffer_objects.c
>>> @@ -342,6 +342,7 @@ brw_get_buffer_subdata(struct gl_context *ctx,
>>>
>>>      unsigned int map_flags = MAP_READ;
>>>      mem_copy_fn memcpy_fn = memcpy;
>>> +#ifdef USE_SSE4_1
>>
>>
>> I don't think anything is defining USE_SSE4_1 in mesa, so you are
>> disabling this code for everyone.
>
>
> Right .. it should be USE_SSE41, r-b with that
>
Hmm you've beat me to it - yes USE_SSE41 it is. With that:
Reviewed-by: Emil Velikov <emil.veli...@collabora.com>

Fwiw: The construct can be seen elsewhere - might want to move to a
helper at some later stage.

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

Reply via email to