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

Jonas Hahnfeld via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 20 05:31:35 PDT 2021


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



More information about the llvm-commits mailing list