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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
