[libcxx-commits] [PATCH] D117357: [libc++] Add the "lint" scripts to libcxx-generate-files.

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Jan 18 11:00:49 PST 2022


Mordante added a comment.

I prefer to also have a cmake target. I agree it feels somewhat odd to have the lint in the generate step. Having the `libcxx-lint` target not part of `libcxx-generate-files` would avoid that. I expect to run `libcxx-lint` more often than `libcxx-generate-files`.

Once we settle for an approach I would like to see this commit update the developer documentation mentioning the new target.
Adding a CI job should be done in a separate commit, which I wouldn't mind to do.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117357



More information about the libcxx-commits mailing list