[PATCH] D67719: [clang] [Basic] Enable __has_feature(leak_sanitizer)

2019-09-22 Thread Michał Górny via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL372527: [clang] [Basic] Enable __has_feature(leak_sanitizer) (authored by mgorny, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://

[PATCH] D67719: [clang] [Basic] Enable __has_feature(leak_sanitizer)

2019-09-22 Thread Petr Hosek via Phabricator via cfe-commits
phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67719/new/ https://reviews.llvm.org/D67719 ___ cfe-commits mailing list cfe-commits@l

[PATCH] D67719: [clang] [Basic] Enable __has_feature(leak_sanitizer)

2019-09-20 Thread Michał Górny via Phabricator via cfe-commits
mgorny updated this revision to Diff 221155. mgorny added a comment. Test added. Thanks for the suggestion! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67719/new/ https://reviews.llvm.org/D67719 Files: clang/include/clang/Basic/Features.def clang/test/Lexer/has_feature_leak_sanit

[PATCH] D67719: [clang] [Basic] Enable __has_feature(leak_sanitizer)

2019-09-20 Thread Petr Hosek via Phabricator via cfe-commits
phosek requested changes to this revision. phosek added a comment. This revision now requires changes to proceed. Actually, can you please add a test akin to https://github.com/llvm/llvm-project/blob/master/clang/test/Lexer/has_feature_memory_sanitizer.cpp? CHANGES SINCE LAST ACTION https://r

[PATCH] D67719: [clang] [Basic] Enable __has_feature(leak_sanitizer)

2019-09-20 Thread Petr Hosek via Phabricator via cfe-commits
phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67719/new/ https://reviews.llvm.org/D67719 ___ cfe-commits mailing list cfe-commits@l

[PATCH] D67719: [clang] [Basic] Enable __has_feature(leak_sanitizer)

2019-09-20 Thread Kamil Rytarowski via Phabricator via cfe-commits
krytarowski added a comment. Looks good to me but I will leave the final decision to someone else. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67719/new/ https://reviews.llvm.org/D67719 ___ cfe-commits mailing list cfe-commits@lists.llvm.

[PATCH] D67719: [clang] [Basic] Enable __has_feature(leak_sanitizer)

2019-09-18 Thread Michał Górny via Phabricator via cfe-commits
mgorny created this revision. mgorny added reviewers: krytarowski, eugenis, leonardchan, vitalybuka, phosek. Add a 'leak_sanitizer' feature akin to existing '*_sanitizer' features to let programmers switch code paths accounting for leak sanitizers being enabled. https://reviews.llvm.org/D67719