[PATCH] D96482: [DeveloperPolicy] Specify LLVM's license more clearly.

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 04:28:59 PST 2021


tschuett added a comment.

In D96482#2556325 <https://reviews.llvm.org/D96482#2556325>, @kristof.beyls wrote:

> In D96482#2556315 <https://reviews.llvm.org/D96482#2556315>, @tschuett wrote:
>
>> If you would extend the paragraph, you could say:
>> "For reference the SPDX identifier is ..."
>
> Yes, it could be helpful to also explicitly call out the SPDX identifier. However, there is also value in keeping that introductory paragraph as short as possible.
> On balance, I think it's slightly better to not call out the SPDX identifier in that introductory paragraph.
> Maybe there's value in explicitly calling out the SPDX identifiers in one of the later paragraphs?

Agreed. In fact, I could neither find a WITH nor a SPDX in the whole file.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96482



More information about the llvm-commits mailing list