[PATCH] D79236: [docs] Regenerate DiagnosticsReference.rst

Mon May 4 18:50:45 PDT 2020

rjmccall added a comment.

In D79236#2019184 <https://reviews.llvm.org/D79236#2019184>, @rsmith wrote:

> In D79236#2019005 <https://reviews.llvm.org/D79236#2019005>, @rjmccall wrote:
>
> > Okay.  So it doesn't just run `ninja docs-clang-html`?  That's unfortunate.
>
>
> 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.

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.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79236