aidengrossman added a comment. In D149809#4317610 <https://reviews.llvm.org/D149809#4317610>, @tstellar wrote:
> I think we could remove some of the duplication by making the docs_target > parameter into a list and passing both docs-clang-html and docs-clang-man to > it. If I'm understanding things correctly we wouldn't be able to do this for either the `gen_docs_depends` or `gen_rst_file_from_td` functions because someone might have the HTML docs build turned on and the man pages build turned off or vice versa. We can only add dependencies to the target after it's created as far as I'm aware. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149809/new/ https://reviews.llvm.org/D149809 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
