Skip to content
  • Ilya Biryukov's avatar
    Disable tidy checks with too many hits · d226bc83
    Ilya Biryukov authored
    Summary:
    Some tidy checks have too many hits in the codebase, making it hard to spot
    other results from clang-tidy, therefore rendering the tool less useful.
    
    Two checks were disabled:
      - misc-non-private-member-variable-in-classes in the whole LLVM monorepo,
        it is very common to have those in LLVM and the style guide does not forbid
        them.
      - readability-identifier-naming in the clang subtree. There are thousands of
        violations in 'Sema.h' alone.
    
    Before the change, 'Sema.h' had >1000 tidy warnings, after the change the number
    dropped to 3 warnings (unterminated namespace comments).
    
    Reviewers: alexfh, hokein
    
    Reviewed By: hokein
    
    Subscribers: llvm-commits, cfe-commits
    
    Tags: #clang, #llvm
    
    Differential Revision: https://reviews.llvm.org/D57573
    
    llvm-svn: 352862
    d226bc83
Loading