[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