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

[Michał Górny via llvm-commits]

mgorny added a comment.

In https://reviews.llvm.org/D23755#540172, @beanz wrote:

> @mgorny, I'd rather we go with the parallel solution. While it does use more CPU time, it doesn't adversely impact overall performance, and it doesn't artificially constrain the build graph.


Are you sure about that? It doesn't impact performance much on dual-core system but on a single core it will run twice as long.

Why are you concerned about the build graph? I still think it's kinda logical, though the dependency is more on a side effect of running one of those commands (-> cache file) rather than the actual result. I don't know if that could be expressed more gracefully in CMake.


https://reviews.llvm.org/D23755