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

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 15:49:11 PDT 2017


Nice catch, I didn't even think about that (who thinks about Hurd these
days?)

Seems reasonable to make this a __GNU__ and  __LINUX__ check.

Thanks!


On Mon, May 15, 2017 at 3:46 PM Kamil Rytarowski via Phabricator <
reviews at reviews.llvm.org> wrote:

> krytarowski added a comment.
>
> `__GNU__` is for GNU/Hurd not, GNU compiler, please revert
>
> We don't want this on NetBSD.
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D33211
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170515/c06b35e6/attachment.html>


More information about the llvm-commits mailing list