[PATCH] D140880: Disable PIE

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 20 09:11:31 PST 2023


lebedev.ri added a comment.

In D140880#4069399 <https://reviews.llvm.org/D140880#4069399>, @foad wrote:

>> That being said, i don't think this can be just set, at best it should be hidden under an opt-in flag.
>
> I don't think there's much point doing this at all if it's opt-in. If you already know that you're suffering from a slow development cycle due to PIE, you might as well just type `cmake -DCMAKE_EXE_LINKER_FLAGS=-no-pie`. The idea of doing by default was to fix the problem for developers who didn't even know they had the problem.

Right. I get that. The problem is that this *is* going to affect everyone,
including distributions, that wanted PIE to be the default in the first place.
So once again, i don't think that's generally viable.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140880/new/

https://reviews.llvm.org/D140880



More information about the llvm-commits mailing list