On 30/06/17 22:22, Jelle van der Waa wrote: > On 06/30/17 at 09:49pm, Evangelos Foutras via arch-dev-public wrote: >> On 30 June 2017 at 18:56, Daniel Micay via arch-dev-public >> <[email protected]> wrote: >>> It's probably a good idea to leave it in CFLAGS for Clang. >> >> I am planning to patch Clang to follow GCC's behavior similarly to >> what Alpine does. [1] >> >> We can discuss dropping the related compilation flags from >> makepkg.conf at a later stage. >> >> [1] https://git.alpinelinux.org/cgit/aports/tree/main/clang > > Seeing that Arch is defined, will you upstream these changes? If so (+1)
I don't think special-casing distros is the right way to go for PIE and SSP; I would much rather see upstream add a CMake option for those. [1] I'll probably patch isPIEDefault() and GetDefaultStackProtectorLevel() to return true and 2 respectively; assuming it actually works and I am able to address any test failures. [1] https://bugs.llvm.org/show_bug.cgi?id=13410
