[PATCH] D21962: MPITypeMismatchCheck for Clang-Tidy

Alexander Droste via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 21 16:08:23 PDT 2016


Alexander_Droste updated this revision to Diff 64985.
Alexander_Droste added a comment.

- use `llvm::StringMap` instead of `std::map`
- `getQualifiedNameAsString` -> `getName`
- remove redundant map lookup
- create MPIi module
- replace `misc` with  `mpi` within the check

I created an MPI module based on how `misc` is set up. Somehow the 
`mpi-type-mismatch` is not found when the tests are run or when I try to 
apply the check on a project but I am assuming you see at first glance what
ingredient missing.


https://reviews.llvm.org/D21962

Files:
  clang-tidy/CMakeLists.txt
  clang-tidy/mpi/CMakeLists.txt
  clang-tidy/mpi/MPITidyModule.cpp
  clang-tidy/mpi/TypeMismatchCheck.cpp
  clang-tidy/mpi/TypeMismatchCheck.h
  clang-tidy/plugin/CMakeLists.txt
  clang-tidy/tool/CMakeLists.txt
  docs/clang-tidy/checks/list.rst
  docs/clang-tidy/checks/mpi-type-mismatch.rst
  test/clang-tidy/Inputs/mpi-type-mismatch/mpimock.h
  test/clang-tidy/mpi-type-mismatch.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21962.64985.patch
Type: text/x-patch
Size: 33473 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160721/c93b9ddd/attachment-0001.bin>


More information about the cfe-commits mailing list