[PATCH] D110261: [libc++][release] Do not force building the runtimes with -fPIC

Louis Dionne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 22 11:12:49 PDT 2021


ldionne added a comment.

In D110261#3015789 <https://reviews.llvm.org/D110261#3015789>, @JamesNagurne wrote:

> Adding relevant people from my team.
>
> I fear that while this change may make people from point 5 happy, it will add a point 6 to the summary where people are unhappy about the default ABI change.

FWIW, we've had that ABI (no `-fPIC` by default) since LLVM 9. We changed it back to having `-fPIC` by default in LLVM 13 (which is being released right now). If this goes through, I would cherry-pick this on the LLVM 13 release branch ASAP so that we don't flip-flop our users too much.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110261



More information about the llvm-commits mailing list