[PATCH] D87085: Add flang out of tree buildbot

Galina via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 8 15:39:39 PDT 2020


gkistanova requested changes to this revision.
gkistanova added a comment.
This revision now requires changes to proceed.

Thanks for working on this, Diana!

It does not seem `getFlangOutOfTreeBuildFactory` belongs to the `UnifiedTreeBuilder`.

You either need to teach cmake to configure flang from the root of the unified tree; then you likely would not need a custom build factory, as the current `UnifiedTreeBuilder` would build your configuration just fine.
Or you need to add a custom builder for flang.

I personally prefer the first option, since it would simplify the build of this configuration in general for everyone, not just for CI.


Repository:
  rZORG LLVM Github Zorg

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

https://reviews.llvm.org/D87085



More information about the llvm-commits mailing list