This is an archive of the discontinued LLVM Phabricator instance.

[Driver] Add SafeStack to a map of incompatible sanitizers
ClosedPublic

Authored by phosek on Feb 21 2018, 8:34 PM.

Diff Detail

Repository
rC Clang

Event Timeline

phosek created this revision.Feb 21 2018, 8:34 PM
vitalybuka accepted this revision.Feb 25 2018, 2:23 PM
vitalybuka added inline comments.
clang/lib/Driver/SanitizerArgs.cpp
384

I guess we check only one direction.
E.g. we have std::make_pair(Address, Thread but no std::make_pair(Thread, Address

So either this line or changes above are not needed.

This revision is now accepted and ready to land.Feb 25 2018, 2:23 PM
phosek updated this revision to Diff 136002.Feb 26 2018, 3:45 PM
phosek marked an inline comment as done.
Herald added subscribers: Restricted Project, llvm-commits. · View Herald TranscriptFeb 26 2018, 3:45 PM
This revision was automatically updated to reflect the committed changes.