[PATCH] D33211: [builtins] Fix a check from __GNU__ to __GNUC__ for disabling executable stack.

Manoj Gupta via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 13:42:21 PDT 2017


manojgupta created this revision.
Herald added subscribers: krytarowski, srhines.

Neither GCC nor Clang define __GNU__. Instead use __GNUC__
for the check.


https://reviews.llvm.org/D33211

Files:
  lib/builtins/assembly.h


Index: lib/builtins/assembly.h
===================================================================
--- lib/builtins/assembly.h
+++ lib/builtins/assembly.h
@@ -44,7 +44,7 @@
 #endif
 #define CONST_SECTION .section .rodata
 
-#if defined(__GNU__) || defined(__ANDROID__) || defined(__FreeBSD__)
+#if defined(__GNUC__) || defined(__ANDROID__) || defined(__FreeBSD__)
 #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: D33211.99058.patch
Type: text/x-patch
Size: 475 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170515/b212f534/attachment.bin>


More information about the llvm-commits mailing list