[PATCH] D54943: [clang-tidy] implement const-transformation for cppcoreguidelines-const-correctness

Florin Iucha via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 31 14:05:08 PST 2019


Hi Jonas,


Don't know how to attach another patch to Phabricator, but I have added 
some tests that fail.


Happy New Year!

florin

On 12/31/19 10:11 AM, Jonas Toth via Phabricator wrote:
> JonasToth updated this revision to Diff 235712.
> JonasToth added a comment.
>
> - fix error from macro-defined variables
> - move test files in sub-directory
> - fix false positive of news in type-dependent contextes
> - add a true positive in typedependent context
> - update to latest utility version
>
>
> Repository:
>    rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>    https://reviews.llvm.org/D54943/new/
>
> https://reviews.llvm.org/D54943
>
> Files:
>    clang-tools-extra/clang-tidy/cppcoreguidelines/CMakeLists.txt
>    clang-tools-extra/clang-tidy/cppcoreguidelines/ConstCorrectnessCheck.cpp
>    clang-tools-extra/clang-tidy/cppcoreguidelines/ConstCorrectnessCheck.h
>    clang-tools-extra/clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModule.cpp
>    clang-tools-extra/clang-tidy/performance/ForRangeCopyCheck.cpp
>    clang-tools-extra/clang-tidy/performance/UnnecessaryCopyInitialization.cpp
>    clang-tools-extra/clang-tidy/performance/UnnecessaryValueParamCheck.cpp
>    clang-tools-extra/clang-tidy/utils/FixItHintUtils.cpp
>    clang-tools-extra/clang-tidy/utils/FixItHintUtils.h
>    clang-tools-extra/clang-tidy/utils/LexerUtils.cpp
>    clang-tools-extra/clang-tidy/utils/LexerUtils.h
>    clang-tools-extra/docs/ReleaseNotes.rst
>    clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines-const-correctness.rst
>    clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-const-correctness-pointer-as-values.cpp
>    clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-const-correctness-transform-pointer-as-values.cpp
>    clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-const-correctness-transform-values.cpp
>    clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-const-correctness-values.cpp
>    clang-tools-extra/unittests/clang-tidy/AddConstTest.cpp
>    clang-tools-extra/unittests/clang-tidy/CMakeLists.txt
>    clang/lib/Analysis/ExprMutationAnalyzer.cpp
>    clang/unittests/Analysis/ExprMutationAnalyzerTest.cpp
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-extra-tests.patch
Type: text/x-patch
Size: 4081 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191231/e2c73f3e/attachment.bin>


More information about the cfe-commits mailing list