[Openmp-commits] [PATCH] D108404: [libomptarget][amdcgn] Add build dependency for llvm-link and opt

[Jonas Hahnfeld via Phabricator via Openmp-commits]

Hahnfeld added a comment.

In D108404#2957095 <https://reviews.llvm.org/D108404#2957095>, @JonChesterfield wrote:

> Reading the ninja file to determine what cmake's dependency graph looked like is a great idea. Going to use that in future, thanks for the comment!

(drive-by comment unrelated to the patch, but because of "reading ninja files")

You know about Ninja's "Extra Tools" <https://ninja-build.org/manual.html#_extra_tools>, in particular `ninja -t browse` and `ninja -t graph`? That should be a nicer UX than going through the machine-readable `build.ninja` 😉


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108404