Re: [PATCH] D13122: Enable SafeStack on all Linux platforms

2015-09-24 Thread Alexey Samsonov via cfe-commits
samsonov accepted this revision. samsonov added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM http://reviews.llvm.org/D13122 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D13122: Enable SafeStack on all Linux platforms

2015-09-24 Thread Evgeniy Stepanov via cfe-commits
eugenis added a comment. Thanks, committed as r248518 with a test. Repository: rL LLVM http://reviews.llvm.org/D13122 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D13122: Enable SafeStack on all Linux platforms

2015-09-23 Thread Evgeniy Stepanov via cfe-commits
eugenis created this revision. eugenis added reviewers: samsonov, pcc. eugenis added a subscriber: cfe-commits. eugenis set the repository for this revision to rL LLVM. I don't see the point in limiting it to x86/x86_64 in the driver. It is only slightly less broken on x86 than on other