[Mesa-dev] [PATCH 2/9] radeonsi: clean up more HAVE_LLVM #ifdefs

Marek Olšák maraeo at gmail.com
Fri Jan 6 16:29:16 UTC 2017


On Fri, Jan 6, 2017 at 12:42 PM, Nicolai Hähnle <nhaehnle at gmail.com> wrote:
> On 02.01.2017 21:16, Marek Olšák wrote:
>>
>> From: Marek Olšák <marek.olsak at amd.com>
>>
>> ---
>>  src/gallium/drivers/radeon/r600_pipe_common.c | 14 +++++++++-----
>>  src/gallium/drivers/radeonsi/si_shader.c      | 19 +++++++++++--------
>>  2 files changed, 20 insertions(+), 13 deletions(-)
>>
>> diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c
>> b/src/gallium/drivers/radeon/r600_pipe_common.c
>> index 74e8de9..d45a385 100644
>> --- a/src/gallium/drivers/radeon/r600_pipe_common.c
>> +++ b/src/gallium/drivers/radeon/r600_pipe_common.c
>> @@ -36,20 +36,24 @@
>>  #include "vl/vl_decoder.h"
>>  #include "vl/vl_video_buffer.h"
>>  #include "radeon/radeon_video.h"
>>  #include <inttypes.h>
>>  #include <sys/utsname.h>
>>
>>  #ifndef HAVE_LLVM
>>  #define HAVE_LLVM 0
>>  #endif
>>
>> +#ifndef MESA_LLVM_VERSION_PATCH
>> +#define MESA_LLVM_VERSION_PATCH 0
>> +#endif
>> +
>
>
> Are you sure this isn't needed? configure.ac looks like it doesn't set this
> if only r600 without llvm is compiled.

I don't understand what you are asking. :) I think it *is* needed.

Marek


More information about the mesa-dev mailing list