[PATCH] D48446: [ubsan] Add support for reporting diagnostics to a monitor process

Kuba (Brecka) Mracek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 21 11:20:04 PDT 2018


kubamracek added a comment.

Thanks, Vedant! This looks good to me.

There's some tests in LLDB, https://github.com/llvm-mirror/lldb/tree/master/packages/Python/lldbsuite/test/functionalities/ubsan that are testing this. But it should be possible to write a test in compiler-rt as well, see https://github.com/llvm-mirror/compiler-rt/blob/master/test/tsan/Darwin/debug_external.cc. But if that turns out to be too much work, I'm okay with just the LLDB test.


https://reviews.llvm.org/D48446





More information about the llvm-commits mailing list