[Mesa-dev] [PATCH] configure.ac: Drop LLVM compiler flags more radically

[Michel Dänzer]

On 09/02/17 10:50 PM, Emil Velikov wrote:
> On 9 February 2017 at 08:07, Michel Dänzer <michel at daenzer.net> wrote:
>> From: Michel Dänzer <michel.daenzer at amd.com>
>>
>> Drop all -m*, -W*, -O*, -g* and -f* flags, with the exception of
>> -fno-rtti, which must be used if it's part of the llvm-config --cxxflags
>> output. We don't want LLVM to dictate the flags we use, and it can even
>> cause build failures, e.g. if LLVM and Mesa are built with different
>> compilers.
>>
> Yes, please !
> Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

Thanks!


> Out of curiosity:
> Are you speaking of personal experience ? What was stored in the
> c/cpp/cxx flags that triggered build failure ?

Building LLVM with clang 4.0 resulted in llvm-config --cxxflags
containing -Wstring-conversion and -fcolor-diagnostics, which aren't
supported by gcc 6.3.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer