[Parallel_libs-commits] [PATCH] D27256: [ARM] Add ARM contributions agreement.

[Francesco Petrogalli via Phabricator via Parallel_libs-commits]

fpetrogalli added a comment.

In https://reviews.llvm.org/D27256#609000, @rengolin wrote:

> In https://reviews.llvm.org/D27256#608999, @fpetrogalli wrote:
>
> > - LLVM: http://llvm.org/svn/llvm-project/llvm/trunk/lib/Target/ARM/LICENSE.TXT (see table at the end of http://llvm.org/svn/llvm-project/llvm/trunk/LICENSE.TXT)
>
>
> If that's there already, why add a new one on the root of the project?


Isn't the license relative to the source tree? So that each of clang/llvm/openmp/paralell-libs/whatever-llvm subproject could have in principle different licenses?

All I have done here is to apply a request from our legal team, who has been asking to add such extension to the code base, as a requirement for any contribution ARM intends to do to parallel libs.

I see your point though: 1) parallel-libs is an llvm subproject which (quoting from README.rst) "[...] will use the normal LLVM license [...]"  and 2) an "ARM contributions agreement" is already stated in the license file of the LLVM codebase, therefore 3) parallel-libs inherits the ARM contribution extension.

At the same time, I am not a lawyer, so I trust that our legal team knows what is best to do here.

One last thing. The contributions that ARM intend to upload to parallel-libs could in principle make parallel-libs an external (to llvm) library so that I somehow see the point of making sure that any contribution outside of an llvm context is covered by ARM contribution agreement (disclaimer: this last sentence is my personal interpretation of the problem, take it with a pinch of salt, i.e. don't base your decision on this as I might have misinterpreted the legal need of this license extension)

Thanks!


https://reviews.llvm.org/D27256