This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Add ARCHS option to add_sanitizer_rt_symbols.
ClosedPublic

Authored by beanz on Aug 27 2015, 11:47 AM.

Details

Summary

This is another step in a multi-step refactoring to move add_sanitizer_rt_symbols in the direction of other add_* functions in compiler-rt.

Changes to CMakeLists files are all minimal except ubsan which tests the new ARCHS loop.

Further cleanup patches will follow.

Diff Detail

Event Timeline

beanz updated this revision to Diff 33343.Aug 27 2015, 11:47 AM
beanz retitled this revision from to [CMake] Add ARCHS option to add_sanitizer_rt_symbols..
beanz updated this object.
beanz added a subscriber: llvm-commits.
samsonov accepted this revision.Aug 27 2015, 1:03 PM
samsonov edited edge metadata.

LGTM (with a nit).

cmake/Modules/SanitizerUtils.cmake
7

Target name will now be <name>-<arch>-symbols.

This revision is now accepted and ready to land.Aug 27 2015, 1:03 PM
This revision was automatically updated to reflect the committed changes.