MaskRay added inline comments.
================ Comment at: clang/CMakeLists.txt:230 +option(CLANG_DEFAULT_PIE "Default to -fPIE and -pie (Linux only)" OFF) +if(CLANG_DEFAULT_PIE) ---------------- joerg wrote: > This option should really be called something like CLANG_DEFAULT_PIE_ON_LINUX > or so. It should be explicit from the name that it only effects that > toolchain and not all the various toolchains that support PIE. This name is fine with me. This ecosystem variety probably only happens for Linux and other operations systems probably can just switch unconditionally (with possibly gated on an os version). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113372/new/ https://reviews.llvm.org/D113372 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits