[clang] modified AST for SEI redemption project (PR #111705)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 17 05:39:08 PDT 2024
================
@@ -11,6 +11,22 @@
// similar to RecursiveASTVisitor.
//
//===----------------------------------------------------------------------===//
+//
+// Modifications to this file by SEI staff are copyright Carnegie Mellon
+// University and contributed under the Apache License v2.0 with LLVM
+// Exceptions.
+//
+// SEI Contributions are made with funding sand support from the Department of
+// Defense under Contract No. FA8702-15-D-0002 with Carnegie Mellon University
+// for the operation of the Software Engineering Institute, a federally funded
+// research and development center.
+//
+// The view, opinions, and/or findings contained in this material are those of
+// the author(s) and should not be construed as an official Government position,
+// policy, or decision, unless designated by other documentation.
+// DM24-0194
----------------
AaronBallman wrote:
We don't accept these kinds of comments in the source as 1) the copyright part is not necessary per https://llvm.org/docs/DeveloperPolicy.html#copyright-license-patents, 2) it doesn't scale well because the files will quickly become littered with such notices, 3) I think anything like this would require LLVM Foundation approval.
Is it a requirement for your contract to have these kinds of comments in the source code? Or would putting some of it (not the copyright paragraph) into the commit message when landing the changes suffice?
https://github.com/llvm/llvm-project/pull/111705
More information about the cfe-commits
mailing list