[PATCH] D73521: [analyzer] add-new-checker.py: Introduction

Csaba Dabis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 30 20:46:07 PDT 2020


Charusso added a comment.

In D73521#1923693 <https://reviews.llvm.org/D73521#1923693>, @NoQ wrote:

> Or is each test run updating the repo? Can we simply make the script do `cat DummyChecker.cpp | sed s/DummyChecker/$NAME/g > $NAME.cpp` and store the checker only once?


It was updating, yes. The first idea was to copy over the live checker, but I wanted to make it as portable as possible.



================
Comment at: clang/utils/analyzer/add-new-checker.py:714-715
+                        help='the path to llvm-tblgen')
+    parser.add_argument('--force-creation', dest='is_force_creation',
+                        action='store_true', help=argparse.SUPPRESS)
+    parser.add_argument('--test', dest='is_test',
----------------
NoQ wrote:
> So, `--overwrite` and add help?
It was only made for the test. Removed.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73521/new/

https://reviews.llvm.org/D73521





More information about the cfe-commits mailing list