[PATCH] D28265: disable sigaltstack on Apple platforms

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 4 10:29:49 PST 2017


davide added inline comments.


================
Comment at: config-ix.cmake:170-171
 # AddressSanitizer conflicts with lib/Support/Unix/Signals.inc
-if( HAVE_SIGNAL_H AND NOT LLVM_USE_SANITIZER MATCHES ".*Address.*")
+# and backtraces do not work on Apple platforms when using sigaltstack.
+if( HAVE_SIGNAL_H AND NOT LLVM_USE_SANITIZER MATCHES ".*Address.*" AND NOT APPLE )
   check_symbol_exists(sigaltstack signal.h HAVE_SIGALTSTACK)
----------------
davide wrote:
> Just a small request: can you you expand this comment a little more? A condensed version of your comment would be excellent, ideally we should have a reference to something publicly accessing, but, that lacking a reference to the rdar is probably fine).
*publicly accessible


https://reviews.llvm.org/D28265





More information about the llvm-commits mailing list