aaron.ballman added inline comments.
================ Comment at: clang/docs/ClangCommandLineReference.rst:2-5 ------------------------------------------------------------------- NOTE: This file is automatically generated by running clang-tblgen -gen-opt-docs. Do not edit this file by hand!! ------------------------------------------------------------------- ---------------- mizvekov wrote: > aaron.ballman wrote: > > I think you might have missed this bit. ;-) > > > > @jansvoboda11 -- do you know of an ergonomic way for people to associate > > extra documentation along with the command line reference to support adding > > documentation like this? This is the second time in the past hour I've seen > > people adding documentation directly to this file and the docs would be > > kind of onerous to put in `HelpText<>` directly. I'm wondering if the FE > > options can support something along the lines of AttrDocs.td where we can > > specify some out-of-line documentation in another .td for things that are > > nontrivial to document? > Ooops! > > Why do we even commit this file then? :-) +1 -- we automatically generate the document from the cmake sphinx target, so I think it's reasonable to remove this file from version control (same as we did for AttributeReference.rst and DiagnosticReference.rst). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133209/new/ https://reviews.llvm.org/D133209 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits