[Openmp-commits] [PATCH] D101509: An attempt to abandon omptarget out-of-tree builds.

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield added a comment.

In D101509#2725126 <https://reviews.llvm.org/D101509#2725126>, @mgorny wrote:

> In D101509#2725078 <https://reviews.llvm.org/D101509#2725078>, @JonChesterfield wrote:
>
>> In D101509#2724846 <https://reviews.llvm.org/D101509#2724846>, @mgorny wrote:
>>
>>> Is this going to block all direct builds of openmp or just these that don't have an LLVM source tree handy?
>>
>> This is aimed at libomptarget, libopenmp can still build without llvm, provided it excludes the target part.
>
> This doesn't really answer my question.

Perhaps you could rephrase the question? This patch is intended to leave openmp building exactly like it used to. The subcomponent libomptarget would no longer build without access to the llvm libraries, but that subcomponent is optional.

In D101509#2725137 <https://reviews.llvm.org/D101509#2725137>, @protze.joachim wrote:

> For me, the main use-case for stand-alone builds is runtime code development. In that case, I use a ("nightly") build of LLVM and use that to compile and test stand-alone runtime builds. 
> Switching branches regularly results in recompiling a lot of code. With stand-alone runtime builds, this is restricted to OpenMP runtime code.
>
> If you really only depend on LLVM libraries and header files, they should get installed. find_package(LLVM) should provide you all necessary information to include and link those libraries.

That is interesting. The header files used by libomptarget may not be on the installed list, but perhaps they should be. For efficient development purposes, using headers from the source tree with binaries from somewhere on disk would be great, provided that slight mashup works. I've been building llvm from scratch repeatedly, changing to a find_package workflow would do good things for iteration time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101509