[Mesa-dev] [PATCH 1/7] intel: Move clflush helpers from anv to common/gen_clflush.h.

[Kenneth Graunke]

On Friday, July 14, 2017 10:09:10 AM PDT Rob Herring wrote:
> On Wed, Jul 5, 2017 at 3:56 PM, Kenneth Graunke <kenneth at whitecape.org> wrote:
> > I want to use these in the OpenGL driver as well.
> 
> FYI, this breaks building the i965 driver on non-x86 builds. Maybe my
> Android CI job should just stop doing that, but it did work before
> this.
> 
> Rob
> 
> >
> > Cc: Jason Ekstrand <jason at jlekstrand.net>
> > ---
> >  src/intel/common/gen_clflush.h     | 56 ++++++++++++++++++++++++++++++++++++++
> >  src/intel/vulkan/anv_batch_chain.c |  2 +-
> >  src/intel/vulkan/anv_device.c      |  2 +-
> >  src/intel/vulkan/anv_private.h     | 32 ++--------------------
> >  src/intel/vulkan/anv_queue.c       |  2 +-
> >  src/intel/vulkan/genX_blorp_exec.c |  2 +-
> >  6 files changed, 62 insertions(+), 34 deletions(-)
> >  create mode 100644 src/intel/common/gen_clflush.h
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 

Sorry about that...I hadn't realized it would be a problem :(

We fixed this for the automake build in commits 6b05c080f202c25531f5 and
5ffe0c9e1be0257de19431 by adding -msse2 to CFLAGS.  (All CPUs that can be
paired with G965 GPUs support SSE2, apparently, so this shouldn't be a
problem.)

We probably need to add that to Android.mk as well...
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170715/28b08627/attachment.sig>