On Wed, Jul 13, 2016 at 10:21 AM, Nicolai Hähnle <nhaeh...@gmail.com> wrote:
> On 13.07.2016 00:56, Marek Olšák wrote:
>>
>> From: Marek Olšák <marek.ol...@amd.com>
>>
>> Not sure if this is the right way to do it, but it seems to work.
>>
>> v2: make it a no-op on LLVM <= 3.5
>> ---
>> src/gallium/auxiliary/gallivm/lp_bld_misc.cpp | 11 +++++++++++
>> src/gallium/auxiliary/gallivm/lp_bld_misc.h | 3 +++
>> 2 files changed, 14 insertions(+)
>>
>> diff --git a/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp
>> b/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp
>> index 23ef3ed..791a470 100644
>> --- a/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp
>> +++ b/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp
>> @@ -697,3 +697,14 @@ lp_free_memory_manager(LLVMMCJITMemoryManagerRef
>> memorymgr)
>> {
>> delete reinterpret_cast<BaseMemoryManager*>(memorymgr);
>> }
>> +
>> +extern "C" void
>> +lp_add_attr_dereferenceable(LLVMValueRef val, uint64_t bytes)
>
>
> Is the extern "C" here really needed? I think the compiler should pick it up
> from the extern "C" wrapper in the header.
I don't know. Other functions in this file use extern "C" too.
Marek
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
