[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