It looks like it has been a while since the checked-in version of
DiagnosticsReference.rst was regenerated. I realise there probably
isn't any expectation that the checked-in version is kept up-to-date,
but now that the project is on github and the rst can be viewed directly
from the repo's web interface, it seemed worth having something a bit
more recent.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
There's no good reason for this file to exist; documenting the warning options is useful, but the file should be generated as an intermediate output of the documentation build and then ultimately fed into Sphinx.
There's even a TODO in the docs about it:
TODO: Generate this from tblgen. Define one anchor per warning group.
Comment Actions
I think it's preferable to commit the file in its updated state so that the public documentation is kept up to date rather than being left stale. However, we should also work to move this file to be built on demand like the attribute reference already is.
Comment Actions
We do generate this document with tblgen already. The problem is that it's not automatically regenerated as part of the docs build on the website, unlike some of our other similar tblgen-generated documentation. I've asked for that to happen repeatedly for years but so far I have achieved zero traction. So the best we have right now is to manually regenerate this every once in a while and check it in. :-(
Comment Actions
Do we need support from the website ops people on this, or do we just need to set up the build system properly to do it?
Comment Actions
My understanding is that there's a custom script on the website that runs some clang-tblgen invocations as part of the docs build process, and it's not something that we can modify by changing anything that's checked into git. (I mean, we could hack around it by detecting one of the tablegen invocations that's already performed and doing more work on the side, but that's clearly the wrong way to get this to work...)
Comment Actions
It didn't last time I looked into this. And http://clang.llvm.org/docs/DiagnosticsReference.html shows the "before" state of this patch, not the "after", so I think it still doesn't.
Comment Actions
Okay. And I can verify that it is in fact rebuilt by docs-clang-html. So I agree that we should continue taking patches like this in the short term, and maybe we can restart that conversation about the website. It's hard to imagine why we would be able to run clang-tblgen as part of the server build but not a full docs-clang-html build. It's not like clang-tblgen wouldn't still need to be sandboxed.