This is an archive of the discontinued LLVM Phabricator instance.

[ubsan] Remove ubsan_standalone_cxx shared library.
ClosedPublic

Authored by eugenis on Sep 27 2017, 5:07 PM.

Details

Summary

Link everything, including the C++ bits, in the single
ubsan_standalone SHARED library. This matches ASan setup.

Diff Detail

Repository
rL LLVM

Event Timeline

eugenis created this revision.Sep 27 2017, 5:07 PM

Note that Clang driver never links UBSan as a shared library, which means that this configuration is completely untested and also that this change is 100% safe.
This build rule was added in https://reviews.llvm.org/D33216.
I'm working on Android testing for ubsan-standalone that would exercise this library.

eugenis updated this revision to Diff 116909.Sep 27 2017, 5:21 PM

Remove -> Merge.

alekseyshl accepted this revision.Sep 27 2017, 5:22 PM
This revision is now accepted and ready to land.Sep 27 2017, 5:22 PM
This revision was automatically updated to reflect the committed changes.