Skip to content

Conversation

@androm3da
Copy link
Member

When this file was first contributed - 28b01c59c93d ([hexagon] Add {hvx,}hexagon_{protos,circ_brev...}, 2021-06-30) - I incorrectly included a QuIC copyright statement with "All rights reserved". I should have contributed this file with the Apache+LLVM exception license.

When this file was first contributed - `28b01c59c93d ([hexagon] Add
{hvx,}hexagon_{protos,circ_brev...}, 2021-06-30)` - I incorrectly included
a QuIC copyright statement with "All rights reserved".  I should have
contributed this file with the `Apache+LLVM exception` license.
@androm3da androm3da self-assigned this Sep 3, 2024
@llvmbot llvmbot added clang Clang issues not falling into any other category backend:X86 clang:headers Headers provided by Clang, e.g. for intrinsics labels Sep 3, 2024
@llvmbot
Copy link
Member

llvmbot commented Sep 3, 2024

@llvm/pr-subscribers-backend-x86

Author: Brian Cain (androm3da)

Changes

When this file was first contributed - 28b01c59c93d ([hexagon] Add {hvx,}hexagon_{protos,circ_brev...}, 2021-06-30) - I incorrectly included a QuIC copyright statement with "All rights reserved". I should have contributed this file with the Apache+LLVM exception license.


Full diff: https://github.com/llvm/llvm-project/pull/107161.diff

1 Files Affected:

  • (modified) clang/lib/Headers/hexagon_types.h (+8-4)
diff --git a/clang/lib/Headers/hexagon_types.h b/clang/lib/Headers/hexagon_types.h
index 029727cc48176b..8e73fad4bcd424 100644
--- a/clang/lib/Headers/hexagon_types.h
+++ b/clang/lib/Headers/hexagon_types.h
@@ -1,7 +1,11 @@
-/******************************************************************************/
-/*   (c) 2020 Qualcomm Innovation Center, Inc. All rights reserved.           */
-/*                                                                            */
-/******************************************************************************/
+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
 #ifndef HEXAGON_TYPES_H
 #define HEXAGON_TYPES_H
 

@llvmbot
Copy link
Member

llvmbot commented Sep 3, 2024

@llvm/pr-subscribers-clang

Author: Brian Cain (androm3da)

Changes

When this file was first contributed - 28b01c59c93d ([hexagon] Add {hvx,}hexagon_{protos,circ_brev...}, 2021-06-30) - I incorrectly included a QuIC copyright statement with "All rights reserved". I should have contributed this file with the Apache+LLVM exception license.


Full diff: https://github.com/llvm/llvm-project/pull/107161.diff

1 Files Affected:

  • (modified) clang/lib/Headers/hexagon_types.h (+8-4)
diff --git a/clang/lib/Headers/hexagon_types.h b/clang/lib/Headers/hexagon_types.h
index 029727cc48176b..8e73fad4bcd424 100644
--- a/clang/lib/Headers/hexagon_types.h
+++ b/clang/lib/Headers/hexagon_types.h
@@ -1,7 +1,11 @@
-/******************************************************************************/
-/*   (c) 2020 Qualcomm Innovation Center, Inc. All rights reserved.           */
-/*                                                                            */
-/******************************************************************************/
+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
 #ifndef HEXAGON_TYPES_H
 #define HEXAGON_TYPES_H
 

@androm3da
Copy link
Member Author

Note that this was the only source file from the original commit without the correct header.

@androm3da androm3da requested a review from dpalermo September 3, 2024 23:36
@androm3da
Copy link
Member Author

@SundeepKushwaha or @ronlieb could you review this PR? Should be non-controversial I hope.

@androm3da androm3da requested a review from SidManning September 6, 2024 18:01
@SidManning
Copy link
Contributor

LGTM

@androm3da androm3da merged commit 048e46a into llvm:main Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend:X86 clang:headers Headers provided by Clang, e.g. for intrinsics clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants