[PATCH] D80872: [ELF] Set DF_1_PIE for -pie

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 30 12:43:09 PDT 2020


MaskRay marked 2 inline comments as done.
MaskRay added inline comments.


================
Comment at: lld/ELF/SyntheticSections.cpp:1320-1321
     dtFlags1 |= DF_1_NOOPEN;
+  if (config->pie)
+    dtFlags1 |= DF_1_PIE;
   if (config->zNow) {
----------------
emaste wrote:
> should this be in alpha order?
For options which only affect `DF_1_*`, this order is already alphabetical.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80872





More information about the llvm-commits mailing list