[PATCH] D28791: [compiler-rt][crt] Simple crtbegin and crtend implementation

[Chris Bieneman via Phabricator via llvm-commits]

beanz added a subscriber: chandlerc.
beanz added inline comments.


================
Comment at: lib/crt/crtbegin.c:5-6
+ *
+ * This file is dual licensed under the MIT and the University of Illinois Open
+ * Source Licenses. See LICENSE.TXT for details.
+ *
----------------
phosek wrote:
> emaste wrote:
> > As this file would be linked into all executables produced by the toolchain, should it be an exception to the standard LLVM MIT/UIUC license - i.e., 0-clause BSD or CC0?
> > 
> > https://tldrlegal.com/license/bsd-0-clause-license
> > https://creativecommons.org/share-your-work/public-domain/cc0/
> I'd be fine with that.
I've asked @chandlerc to chime in either himself, or someone from the foundation. My understanding of the way we handle licensing in LLVM is that only the runtime libraries are dual licensed UIUC & MIT. The reason being that the MIT license doesn't have the same attribution requirements as UIUC. LLVM, Clang, LLDB, LLD, etc. are all UIUC licensed only.

Since all of compiler-rt is under the UIUC/MIT dual license, I would be very concerned if this file needed different licensing since lib/builtins all get embedded in binaries in the same way, and are under that same license configuration.


Repository:
  rL LLVM

https://reviews.llvm.org/D28791