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://
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
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
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
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
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.
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