[PATCH] D104848: [cmake] Handled utils/unittests before projects

Jakub Chlanda via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 28 08:09:51 PDT 2021


jchlanda added a comment.

In D104848#2910218 <https://reviews.llvm.org/D104848#2910218>, @beanz wrote:

> What are you trying to do in a project that is motivating this?
>
> I don’t have any real objection to this change, but the motivation concerns me. Generally we avoid things that require order-dependence, and setting up target dependencies does not.

This issue was brought forward from Intel's SYCL project, and specifically how it builds `check-sycl` target, you can follow it here: https://github.com/intel/llvm/pull/3964


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104848



More information about the llvm-commits mailing list