This is an archive of the discontinued LLVM Phabricator instance.

[Sanitizers] Add *SAN_OPTIONS to possibly_dangerous_env_vars
ClosedPublic

Authored by filcab on Jun 3 2015, 5:05 PM.

Details

Summary

This way, if they're set when running ninja check-ubsan (or another
sanitizer), they get cleared before we start invoking the programs.

Diff Detail

Repository
rL LLVM

Event Timeline

filcab updated this revision to Diff 27073.Jun 3 2015, 5:05 PM
filcab retitled this revision from to [Sanitizers] Add *SAN_OPTIONS to possibly_dangerous_env_vars.
filcab updated this object.
filcab edited the test plan for this revision. (Show Details)
filcab added reviewers: samsonov, kcc.
filcab added a subscriber: Unknown Object (MLST).
samsonov accepted this revision.Jun 3 2015, 5:13 PM
samsonov edited edge metadata.

OK, let's try this... I've looked at the buildbots and it looks like we're only setting *SAN_OPTIONS environment variables when we're running regular (not sanitizer) tests.

This revision is now accepted and ready to land.Jun 3 2015, 5:13 PM
This revision was automatically updated to reflect the committed changes.