This is an archive of the discontinued LLVM Phabricator instance.

[NewPM][MSan] Add sanitizer at O0
AbandonedPublic

Authored by philip.pfaffe on Feb 5 2019, 3:18 PM.

Details

Summary

New-PM's msan currently is not inserted at O0, unlike with the legacy PM.

Diff Detail

Event Timeline

philip.pfaffe created this revision.Feb 5 2019, 3:18 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 5 2019, 3:18 PM
Herald added a subscriber: bollu. · View Herald Transcript

Please, update the msan test that currently runs with -O1.

leonardchan added a comment.EditedFeb 19 2019, 5:36 PM

Will need to be rebased after rC353985. Should this also include tsan, unless this is just for msan specifically?

Will need to be rebased after rC353985. Should this also include tsan, unless this is just for msan specifically?

Yeah, I suspect so. If you want, maybe take this over? Would be great to get this landed.

Will need to be rebased after rC353985. Should this also include tsan, unless this is just for msan specifically?

Yeah, I suspect so. If you want, maybe take this over? Would be great to get this landed.

Made patch D58424 for this.

philip.pfaffe abandoned this revision.Feb 19 2019, 9:20 PM