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

[Vedant Kumar via Phabricator via llvm-commits]

vsk added inline comments.


================
Comment at: lib/ubsan/ubsan_monitor.h:36
+/// that a UB report is available.
+extern "C" SANITIZER_INTERFACE_ATTRIBUTE void __ubsan_on_report(void);
+
----------------
vitalybuka wrote:
> vsk wrote:
> > vitalybuka wrote:
> > > I'd prefer interface consistent with similar one from TSAN:
> > > __tsan_on_report(void *report) {
> > >   __tsan_get_report_loc(report.....
> > >   __tsan_get_report_loc_object_type(report.....
> > > }
> > > 
> > > Also this will avoid having state set by RegisterUndefinedBehaviorReport
> > Point taken, it would be a bit simpler this way. However, we can't make this change because it would break debuggability of code compiled with Xcode 9.
> Does Xcode 9 works for tsan?
Are you asking if IDE diagnostics for tsan issues found in a binary compiled with Xcode 9, work when running under Xcode (9+K)? I'm not sure. Generally we try to make it possible to debug old binaries.


https://reviews.llvm.org/D48446