Re: [PATCH] D13465: Add SafeStack support for test-suite.

2015-10-13 Thread Valérian Rousset via cfe-commits
tharvik added a comment. In http://reviews.llvm.org/D13465#260846, @jroelofs wrote: > In http://reviews.llvm.org/D13465#260628, @tharvik wrote: > > > When running the test suite, it gives a bunch of `undefined reference to > > '__safestack_unsafe_stack_ptr'`. > > See regression results

Re: [PATCH] D13465: Add SafeStack support for test-suite.

2015-10-06 Thread Jonathan Roelofs via cfe-commits
jroelofs added a comment. In http://reviews.llvm.org/D13465#260628, @tharvik wrote: > When running the test suite, it gives a bunch of `undefined reference to > '__safestack_unsafe_stack_ptr'`. > See regression results . Sounds like it's missing the safestack run

Re: [PATCH] D13465: Add SafeStack support for test-suite.

2015-10-06 Thread Jonathan Roelofs via cfe-commits
jroelofs added a subscriber: jroelofs. jroelofs accepted this revision. jroelofs added a reviewer: jroelofs. jroelofs added a comment. This revision is now accepted and ready to land. LGTM http://reviews.llvm.org/D13465 ___ cfe-commits mailing list