[PATCH] D23755: cmake: Add an ordering dep between HTML & man Sphinx targets

Michał Górny via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 11 02:31:09 PDT 2016


mgorny requested a review of this revision.
mgorny added a reviewer: beanz.
mgorny added a comment.
This revision is now accepted and ready to land.

@beanz, one more thing just occurred to me wrt `USES_TERMINAL`. This forces serialization globally while it is only necessary per-docroot (i.e. per `CMAKE_CURRENT_BINARY_DIR`).

So, with the dependency solution you can e.g. build llvm & clang docs in parallel, while the `USES_TERMINAL` solution will unnecessarily serialize that.

Could you then reconsider my original patch?


https://reviews.llvm.org/D23755





More information about the llvm-commits mailing list