[PATCH] D33219: Fix executable stack directive on Linux.
Manoj Gupta via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 15 16:26:44 PDT 2017
manojgupta updated this revision to Diff 99083.
manojgupta added a comment.
Remove check for Android since it is covered by Linux checks.
https://reviews.llvm.org/D33219
Files:
lib/builtins/assembly.h
Index: lib/builtins/assembly.h
===================================================================
--- lib/builtins/assembly.h
+++ lib/builtins/assembly.h
@@ -44,7 +44,8 @@
#endif
#define CONST_SECTION .section .rodata
-#if defined(__GNUC__) || defined(__ANDROID__) || defined(__FreeBSD__)
+#if defined(__GNU__) || defined(__FreeBSD__) || defined(__Fuchsia__) || \
+ defined(__linux__)
#define NO_EXEC_STACK_DIRECTIVE .section .note.GNU-stack,"",%progbits
#else
#define NO_EXEC_STACK_DIRECTIVE
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33219.99083.patch
Type: text/x-patch
Size: 504 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170515/367b0ec8/attachment.bin>
More information about the llvm-commits
mailing list