[PATCH] D30530: [ELF] - Implemented -z text/-znotext

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 07:45:15 PST 2017


ruiu added inline comments.


================
Comment at: ELF/Driver.cpp:614
   Config->ZStackSize = getZOptionValue(Args, "stack-size", 0);
+  Config->ZText = getZOption(Args, "text", "notext");
   Config->ZWxneeded = hasZOption(Args, "wxneeded");
----------------
Do you actually need -z notext? You can just write `hasZOption(Args, "text")` like other options.


https://reviews.llvm.org/D30530





More information about the llvm-commits mailing list