[PATCH] D82659: Fix missing build dependency on omp_gen.

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 1 08:42:10 PDT 2020


simon_tatham added a comment.

In D82659#2125618 <https://reviews.llvm.org/D82659#2125618>, @clementval wrote:

> LGTM. So later the DEPENDS omp_gen that are in clang subdirectories could be removed right?


That seems likely. I'm thinking what I should probably do is to polish up my checking script and try to get it into `llvm/utils` somewhere, and then it would be possible to make that kind of change and be //confident// that it wasn't removing a necessary dependency.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82659





More information about the llvm-commits mailing list