[PATCH] D140880: Disable PIE

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 20 09:26:49 PST 2023


arsenm added a comment.

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

>> There is a point because it requires I know the flag for this, the cmake flag for this, and need to know I should think about this in the first place. This should be an option prominently advertised in the llvm cmake usage documentation and put in the developer guide
>
> My point was that an off-by-default flag would not help you "to know I should think about this in the first place". But it hadn't occurred to me that we could document it, and people would actually read that. :-)

We probably should have an example "this is the set of flags you actually want" there too. I spent way too much time when I'm on a new system figuring out the full set of LLVM_USE_LINKER=lld and the link threads options and all the others you need for fast builds


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