[PATCH] D25472: [ELF] - Implemented -z wxneeded.

[Rui Ueyama via llvm-commits]

ruiu added a comment.

Could you give me a pointer to the OpenBSD-local patch to GNU ld which implements this -z wxneeded feature? I'd like to look at that patch to see if this patch does the same thing.



================
Comment at: ELF/Config.h:129
   bool ZRelro;
+  bool ZWXNeeded;
   DiscardPolicy Discard;
----------------
The variable naming rule here is to camel-case words with "-" as a word separator. Because "wxneeded" contains no hyphens, it should be handled as one word. Thus, `ZWxneeded`.


================
Comment at: ELF/LinkerScript.cpp:1717
                      .Case("PT_GNU_RELRO", PT_GNU_RELRO)
+                     .Case("PT_OPENBSD_WXNEEDED", PT_OPENBSD_WXNEEDED)
                      .Default(-1);
----------------
Sort.


https://reviews.llvm.org/D25472