This is an archive of the discontinued LLVM Phabricator instance.

[clang] [Basic] Enable __has_feature(leak_sanitizer)
ClosedPublic

Authored by mgorny on Sep 18 2019, 9:50 AM.

Details

Summary

Add a 'leak_sanitizer' feature akin to existing '*_sanitizer' features
to let programmers switch code paths accounting for leak sanitizers
being enabled.

Diff Detail

Repository
rL LLVM

Event Timeline

mgorny created this revision.Sep 18 2019, 9:50 AM

Looks good to me but I will leave the final decision to someone else.

phosek accepted this revision.Sep 20 2019, 10:21 AM

LGTM

This revision is now accepted and ready to land.Sep 20 2019, 10:21 AM
This revision now requires changes to proceed.Sep 20 2019, 10:23 AM
mgorny updated this revision to Diff 221155.Sep 20 2019, 11:21 PM

Test added. Thanks for the suggestion!

krytarowski accepted this revision.Sep 21 2019, 3:49 AM
phosek accepted this revision.Sep 22 2019, 1:43 PM

LGTM

This revision is now accepted and ready to land.Sep 22 2019, 1:43 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 22 2019, 1:53 PM