[PATCH] D41567: Add NO_EXEC_STACK_DIRECTIVE to chkstk and chkstk2 sources

Dimitry Andric via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 24 12:59:19 PST 2017


dim added a comment.

In https://reviews.llvm.org/D41567#963799, @martell wrote:

> hmm looking at the #define for NO_EXEC_STACK_DIRECTIVE
>  once the two files have been removed from non windows targets they won't do anything anymore.


Ah yes, you're right, I didn't realize that.  Those won't really be needed then.

> Do you have commit access, it looks good as is?

Yes, I'll commit.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D41567





More information about the llvm-commits mailing list