On 09/02/17 10:50 PM, Emil Velikov wrote: > On 9 February 2017 at 08:07, Michel Dänzer <[email protected]> wrote: >> From: Michel Dänzer <[email protected]> >> >> 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 <[email protected]>
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 _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
