[PATCH] D30783: [asan] Split SIGSEGV/SIGBUS handling so we can handle only one of them and not the other.

[Filipe Cabecinhas via Phabricator via llvm-commits]

filcab added a comment.

In https://reviews.llvm.org/D30783#697231, @kcc wrote:

> what are those platforms and can we have at the very least tests for those platforms? 
>  (Also, would be nice to have tests for linux/mac, but frankly I don't remember how to cause a SIGBUS on linux)


Hi Kostya,

The PS4 needs to handle those signals differently. As of right now, we can't upload the modifications to make ASan work on it, though. I doubt it would be useful to upstream a test with `REQUIRES: ps4`, though.



================
Comment at: test/asan/TestCases/Posix/asan-sigbus.cpp:4
+// RUN: %clangxx_asan -std=c++11 %s -o %t && not %run %t %T/file 2>&1 | FileCheck %s -check-prefix=CHECK -check-prefix=CHECK-BUS
+// RUN: %clangxx_asan -std=c++11 %s -o %t && %env_asan_opts=handle_bug=false not %run %t %T/file 2>&1 | FileCheck %s
+
----------------
kcc wrote:
> don't compile twice. 
> what's handle_bug? 
A bug in the command line. Updating.


https://reviews.llvm.org/D30783