[PATCH] D140880: Disable PIE

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

> 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. :-)

> The problem is that this *is* going to affect everyone, including distributions, that wanted PIE to be the default in the first place.

I accept that there is probably conflict between what developers want and what distributors want. This patch was my way of voting for the developers. :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140880