[PATCH] D61613: [LLD][ELF] Add the -z ifunc-noplt option

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 12 21:30:46 PDT 2019


ruiu added inline comments.


================
Comment at: lld/ELF/Driver.cpp:936
 
+  if (Config->ZText && Config->ZIfuncNoplt)
+    error("the -z text and -z ifunc-noplt options may not be combined");
----------------
Can you move this to `checkOptions`?


================
Comment at: lld/ELF/Driver.cpp:937
+  if (Config->ZText && Config->ZIfuncNoplt)
+    error("the -z text and -z ifunc-noplt options may not be combined");
+
----------------
remove "the"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61613





More information about the llvm-commits mailing list