This revision was automatically updated to reflect the committed changes.
Closed by commit rL295482: [clang-tidy] Add -path option to clang-tidy-diff.py
(authored by ehsan).
Changed prior to commit:
https://reviews.llvm.org/D29806?vs=88879&id=88959#toc
Repository:
rL LLVM
https://reviews.ll
ehsan updated this revision to Diff 88879.
ehsan added a comment.
Addressed the review comments.
https://reviews.llvm.org/D29806
Files:
clang-tidy/tool/clang-tidy-diff.py
test/clang-tidy/clang-tidy-diff.cpp
Index: test/clang-tidy/clang-tidy-diff.cpp
===
ehsan added a comment.
Gentle ping. :-)
https://reviews.llvm.org/D29806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ehsan added a comment.
In https://reviews.llvm.org/D29806#673329, @alexfh wrote:
> What's your use case? Can it be addressed by just forwarding the -p flag to
> clang-tidy?
I just need to pass the full path to the compilation DB to clang-tidy. The
problem is that invoking `clang-tidy-diff.py
ehsan created this revision.
Herald added a subscriber: JDevlieghere.
This flag allows specifying a custom path for the compilation
database. Unfortunately we can't use the -p flag like other
clang-tidy tools because it's already taken.
https://reviews.llvm.org/D29806
Files:
clang-tidy/tool/
This revision was automatically updated to reflect the committed changes.
Closed by commit rL294607: [clang-tidy] Add -quiet option to suppress extra
output (authored by ehsan).
Changed prior to commit:
https://reviews.llvm.org/D29661?vs=87469&id=87842#toc
Repository:
rL LLVM
https://review
This revision was automatically updated to reflect the committed changes.
Closed by commit rL294491: [clang-tidy] Add -extra-arg and -extra-arg-before to
clang-tidy-diff.py (authored by ehsan).
Changed prior to commit:
https://reviews.llvm.org/D29699?vs=87576&id=87673#toc
Repository:
rL LLVM
ehsan added a comment.
This is equivalent to https://reviews.llvm.org/D28334 for clang-tidy-diff.py.
https://reviews.llvm.org/D29699
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ehsan created this revision.
Herald added a subscriber: JDevlieghere.
These flags allow specifying extra arguments to the tool's command
line which don't appear in the compilation database.
https://reviews.llvm.org/D29699
Files:
clang-tidy/tool/clang-tidy-diff.py
Index: clang-tidy/tool/clan
ehsan created this revision.
Herald added a subscriber: JDevlieghere.
This new flag instructs clang-tidy to not output anything
except for errors and warnings. This makes it easier to
script clang-tidy to run as part of external build systems.
https://reviews.llvm.org/D29661
Files:
clang-tid
This revision was automatically updated to reflect the committed changes.
ehsan marked an inline comment as done.
Closed by commit rL292415: [clang-tidy] Add -extra-arg and -extra-arg-before to
run-clang-tidy.py (authored by ehsan).
Changed prior to commit:
https://reviews.llvm.org/D28334?vs=83
ehsan marked an inline comment as done.
ehsan added inline comments.
Comment at: clang-tidy/tool/run-clang-tidy.py:80
+ for arg in extra_arg:
+ start.append('-extra-arg=%s' % arg[0])
+ for arg in extra_arg_before:
alexfh wrote:
> Why arg[0] and not just ar
ehsan created this revision.
ehsan added reviewers: bkramer, alexfh, klimek.
ehsan added a subscriber: cfe-commits.
Herald added a subscriber: JDevlieghere.
These flags allow specifying extra arguments to the tool's command
line which don't appear in the compilation database.
https://reviews.llv
13 matches
Mail list logo
