[PATCH] D87085: Add flang out of tree buildbot

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 01:36:49 PDT 2020


rovka updated this revision to Diff 291509.
rovka edited the summary of this revision.
rovka added a comment.

Hi Galina, thanks for having a look!

I think running cmake from the top level directory would defeat the purpose of what this buildbot is trying to test. The whole point is to be able to build flang using only the flang source tree. I have therefore updated the patch to create a new FlangBuilder, but it still imports some things from the UnifiedTreeBuilder. We could also move those common functions from UnifiedTreeBuilder to Util.py if you think that would be cleaner. Please let me know your thoughts.

Cheers!


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

https://reviews.llvm.org/D87085

Files:
  buildbot/osuosl/master/config/builders.py
  buildbot/osuosl/master/config/slaves.py
  zorg/buildbot/builders/FlangBuilder.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D87085.291509.patch
Type: text/x-patch
Size: 5238 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200914/81bb66c3/attachment.bin>


More information about the llvm-commits mailing list