Re: [Mesa-dev] [PATCH 2/2] clover: Re-order includes in invocation.cpp to fix build

2016-07-20 Thread Francisco Jerez
Tom Stellard writes: > The build was failing because the official CL headers have a few defines, > like: > > \# define cl_khr_gl_sharing 1 > > Which have the same name as some class members of clang's OpenCLOptions class. > If we include the cl headers first, this breaks the build because the me

Re: [Mesa-dev] [PATCH 2/2] clover: Re-order includes in invocation.cpp to fix build

2016-07-20 Thread Vedran Miletić
On 07/20/2016 11:46 AM, Tom Stellard wrote: The build was failing because the official CL headers have a few defines, like: \# define cl_khr_gl_sharing 1 Which have the same name as some class members of clang's OpenCLOptions class. If we include the cl headers first, this breaks the build beca

[Mesa-dev] [PATCH 2/2] clover: Re-order includes in invocation.cpp to fix build

2016-07-20 Thread Tom Stellard
The build was failing because the official CL headers have a few defines, like: \# define cl_khr_gl_sharing 1 Which have the same name as some class members of clang's OpenCLOptions class. If we include the cl headers first, this breaks the build because the member names of this class are replace

Re: [Mesa-dev] [PATCH 2/2] clover: Re-order includes in invocation.cpp to fix build

2016-07-19 Thread Ilia Mirkin
On Tue, Jul 19, 2016 at 11:13 AM, Tom Stellard wrote: > The build was failing because the official CL headers have a few defines, > like: like ... > > Which have the same name as some class members of clang's OpenCLOptions class. > If we include the cl headers first, this breaks the build becau

[Mesa-dev] [PATCH 2/2] clover: Re-order includes in invocation.cpp to fix build

2016-07-19 Thread Tom Stellard
The build was failing because the official CL headers have a few defines, like: Which have the same name as some class members of clang's OpenCLOptions class. If we include the cl headers first, this breaks the build because the member names of this class are replaced by the literal 1. --- .../st