[PATCH] D14970: ELF: Omit PT_GNU_STACK segment if -z execstack is provided.

Joerg Sonnenberger via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 24 15:39:20 PST 2015

joerg accepted this revision.
joerg added a comment.
This revision is now accepted and ready to land.

Thanks, this works for the cases I care about. The refinement of the test case is minor, just trying to make sure both cases are handled consistently.

Comment at: test/ELF/gnustack.s:9
@@ +8,3 @@
+# RW:       Sections [
+# RW-NOT:   Name: .note.GNU-stack
+# RW:       ProgramHeaders [
The intention here is that the notes are dropped, so shouldn't that be checked for the RWX case as well?


More information about the llvm-commits mailing list