This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer] Use %clangxx in objc++ test files
ClosedPublic

Authored by kastiglione on Jul 8 2016, 3:33 PM.

Details

Summary

These test in this change are objc++, but are built using %clang, not %clangxx. The reason this works is the driver has been adding -lc++ for sanitizer enabled builds. By making these tests use %clangxx, they no longer depend on the driver linking to c++. The reason I want to make this change is to re-land http://reviews.llvm.org/D22018, which caused these tests to stop working.

Diff Detail

Event Timeline

kastiglione updated this revision to Diff 63334.Jul 8 2016, 3:33 PM
kastiglione retitled this revision from to [sanitizer] Use %clangxx in objc++ test files.
kastiglione updated this object.
kastiglione added reviewers: samsonov, compnerd.
kastiglione added a subscriber: llvm-commits.
kubamracek accepted this revision.Jul 9 2016, 3:30 AM
kubamracek added a reviewer: kubamracek.
kubamracek edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Jul 9 2016, 3:31 AM
compnerd closed this revision.Jul 9 2016, 2:21 PM
compnerd edited edge metadata.

LGTM; Committed as SVN r274989.