This is an archive of the discontinued LLVM Phabricator instance.

[CMake] zlib FORCE_ON overrides MSan
AbandonedPublic

Authored by phosek on Aug 6 2020, 9:00 PM.

Details

Reviewers
hans
Summary

This restores the behavior removed in 1adc494 where setting
LLVM_ENABLE_ZLIB to FORCE_ON overrides MSan.

Diff Detail

Event Timeline

phosek created this revision.Aug 6 2020, 9:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 6 2020, 9:00 PM
phosek requested review of this revision.Aug 6 2020, 9:00 PM
hans accepted this revision.Aug 7 2020, 10:55 AM

lgtm

I don't know if anyone cares about the msan+zlib case, I just think it's important that FORCE_ON either enables zlib or raises an error. Looks like this patch ensures that.

This revision is now accepted and ready to land.Aug 7 2020, 10:55 AM
phosek abandoned this revision.Aug 8 2020, 12:26 PM

This was combined into D79219.