On 14 April 2015 at 13:32, Marek Olšák <mar...@gmail.com> wrote:
> On Tue, Apr 14, 2015 at 2:19 PM, Emil Velikov <emil.l.veli...@gmail.com> 
> wrote:
>> On 13 April 2015 at 21:06, Marek Olšák <mar...@gmail.com> wrote:
>>> From: Marek Olšák <marek.ol...@amd.com>
>>>
>>> ---
>>>  configure.ac | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/configure.ac b/configure.ac
>>> index 9e8c1d8..6ccf3b4 100644
>>> --- a/configure.ac
>>> +++ b/configure.ac
>>> @@ -2543,6 +2543,7 @@ if test "x$MESA_LLVM" = x1; then
>>>      echo "        LLVM_CFLAGS:     $LLVM_CFLAGS"
>>>      echo "        LLVM_CXXFLAGS:   $LLVM_CXXFLAGS"
>>>      echo "        LLVM_CPPFLAGS:   $LLVM_CPPFLAGS"
>>> +    echo "        LLVM_LDFLAGS:    $LLVM_LDFLAGS"
>> I'm puzzled - why do we need this ? Does commit message (ahem) does
>> not mention either.
>
> I'd like to know what the LLVM path is for the linker. Things can go
> crazy when building 32-bit Mesa on 64-bit and this will help to catch
> incorrect LLVM paths before compilation.
>
Fair enough. With the above note in the commit message:
Reviewed-by: Emil Velikov <emil.l.velil...@gmail.com>

Out of curiosity - which distro is this happening on, are you building
your own multilib llvm ? Don't think I've ever had such issue before.

-Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to