[PATCH] D112926: run-clang-tidy.py analyze unique files only

2021-11-05 Thread Serikzhan via Phabricator via cfe-commits
serkazi added a comment. In D112926#3113220 , @salman-javed-nz wrote: > In D112926#3113206 , @serkazi wrote: > >> This is now accepted, please feel free to merge on my behalf. Thanks. > > What's the full name and

[PATCH] D112926: run-clang-tidy.py analyze unique files only

2021-11-05 Thread Serikzhan via Phabricator via cfe-commits
serkazi added a comment. In D112926#3108140 , @salman-javed-nz wrote: > It looks good to me. I don't have any more comments to add - it is a very > small code change after all. > I have commit access and am happy to commit it on your behalf. > > However

[PATCH] D112926: run-clang-tidy.py analyze unique files only

2021-11-04 Thread Serikzhan via Phabricator via cfe-commits
serkazi added a comment. In D112926#3108140 , @salman-javed-nz wrote: > It looks good to me. I don't have any more comments to add - it is a very > small code change after all. > I have commit access and am happy to commit it on your behalf. > > However

[PATCH] D112926: run-clang-tidy.py analyze unique files only

2021-11-02 Thread Serikzhan via Phabricator via cfe-commits
serkazi added a comment. In D112926#3102417 , @salman-javed-nz wrote: > Why does the compile-commands.json have duplicate entries in the first place? > Would someone do that on purpose? Think of a large project where people -- for various reasons -- do

[PATCH] D112926: run-clang-tidy.py analyze unique files only

2021-11-01 Thread Serikzhan via Phabricator via cfe-commits
serkazi updated this revision to Diff 383959. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112926/new/ https://reviews.llvm.org/D112926 Files: clang-tools-extra/clang-tidy/tool/run-clang-tidy.py Index