[PATCH] D33826: [clang-tidy] avoid pointer cast to more strict alignment check

2017-12-19 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti added a comment. In https://reviews.llvm.org/D33826#935394, @lebedev.ri wrote: > Ping? Hi, I'm sorry I got a bit busy, I'll get back to it next year. Repository: rL LLVM https://reviews.llvm.org/D33826 ___ cfe-commits mailing li

[PATCH] D33826: [clang-tidy] avoid pointer cast to more strict alignment check

2017-09-25 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti updated this revision to Diff 116510. Repository: rL LLVM https://reviews.llvm.org/D33826 Files: clang-tidy/cert/AvoidPointerCastToMoreStrictAlignmentCheck.cpp clang-tidy/cert/AvoidPointerCastToMoreStrictAlignmentCheck.h clang-tidy/cert/CERTTidyModule.cpp clang-tidy/cert

[PATCH] D33826: [clang-tidy] avoid pointer cast to more strict alignment check

2017-09-25 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti added a comment. In https://reviews.llvm.org/D33826#866145, @lebedev.ri wrote: > In https://reviews.llvm.org/D33826#866133, @NorenaLeonetti wrote: > > > In https://reviews.llvm.org/D33826#861226, @aaron.ballman wrote: > > > > > Have you run this check over any large code bases to s

[PATCH] D33826: [clang-tidy] avoid pointer cast to more strict alignment check

2017-09-11 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti updated this revision to Diff 114544. Repository: rL LLVM https://reviews.llvm.org/D33826 Files: clang-tidy/cert/AvoidPointerCastToMoreStrictAlignmentCheck.cpp clang-tidy/cert/AvoidPointerCastToMoreStrictAlignmentCheck.h clang-tidy/cert/CERTTidyModule.cpp clang-tidy/cert

[PATCH] D33826: [clang-tidy] avoid pointer cast to more strict alignment check

2017-09-11 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti marked an inline comment as done. NorenaLeonetti added a comment. In https://reviews.llvm.org/D33826#861226, @aaron.ballman wrote: > Have you run this check over any large code bases to see what the quality of > the diagnostics are? `-checks=-*,cert-exp36-c,modernize-unary-stati

[PATCH] D33826: [clang-tidy] avoid pointer cast to more strict alignment check

2017-09-02 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti updated this revision to Diff 113654. NorenaLeonetti added a comment. Herald added a subscriber: JDevlieghere. Added a test for C++ style cast and modified the docs. Repository: rL LLVM https://reviews.llvm.org/D33826 Files: clang-tidy/cert/AvoidPointerCastToMoreStrictAlignm

[PATCH] D33825: [clang-tidy] signal handler must be plain old function check

2017-09-02 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti updated this revision to Diff 113652. Repository: rL LLVM https://reviews.llvm.org/D33825 Files: clang-tidy/cert/CERTTidyModule.cpp clang-tidy/cert/CMakeLists.txt clang-tidy/cert/SignalHandlerMustBePlainOldFunctionCheck.cpp clang-tidy/cert/SignalHandlerMustBePlainOldFunc

[PATCH] D33825: [clang-tidy] signal handler must be plain old function check

2017-09-02 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti updated this revision to Diff 113651. Herald added a subscriber: JDevlieghere. Repository: rL LLVM https://reviews.llvm.org/D33825 Files: clang-tidy/cert/CERTTidyModule.cpp clang-tidy/cert/CMakeLists.txt clang-tidy/cert/SignalHandlerMustBePlainOldFunctionCheck.cpp clang-

[PATCH] D33825: [clang-tidy] signal handler must be plain old function check

2017-09-02 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti marked 16 inline comments as done. NorenaLeonetti added a comment. In https://reviews.llvm.org/D33825#772688, @aaron.ballman wrote: > This check is an interesting one. The rules around what is signal safe are > changing for C++17 to be a bit more lenient than what the rules are fo

[PATCH] D33826: [clang-tidy] avoid pointer cast to more strict alignment check

2017-06-02 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti created this revision. NorenaLeonetti added a project: clang-tools-extra. Herald added subscribers: xazax.hun, mgorny. Based on CERT-EXP36-C Repository: rL LLVM https://reviews.llvm.org/D33826 Files: clang-tidy/cert/AvoidPointerCastToMoreStrictAlignmentCheck.cpp clang-tidy

[PATCH] D33825: [clang-tidy] signal handler must be plain old function check

2017-06-02 Thread Eniko Donatella Toth via Phabricator via cfe-commits
NorenaLeonetti created this revision. NorenaLeonetti added a project: clang-tools-extra. Herald added subscribers: xazax.hun, mgorny. Based on CERT-MSC54-CPP Repository: rL LLVM https://reviews.llvm.org/D33825 Files: clang-tidy/cert/CERTTidyModule.cpp clang-tidy/cert/CMakeLists.txt cla