https://github.com/Venyla updated
https://github.com/llvm/llvm-project/pull/69693
>From d42a9f963471d6e78584febdacf4c5e99419a3c2 Mon Sep 17 00:00:00 2001
From: Vina Zahnd
Date: Fri, 20 Oct 2023 10:01:54 +0200
Subject: [PATCH] [clangd] Add tweak to inline concept requirements
Co-authored-by: Je
llvmbot wrote:
@llvm/pr-subscribers-clangd
Author: Vina (Venyla)
Changes
# Description
This introduces a new refactoring for concept requirements, inlining simple
require clauses into the template declaration.
Here is an example of what it does:
Before
```cpp
template
https://github.com/Venyla updated
https://github.com/llvm/llvm-project/pull/69693
>From e2bb36994e919fd3a6e559d5ad3ba738d00f299f Mon Sep 17 00:00:00 2001
From: Vina Zahnd
Date: Fri, 20 Oct 2023 10:01:54 +0200
Subject: [PATCH] [clangd] Add tweak to inline concept requirements
Co-authored-by: Je
https://github.com/5chmidti commented:
Please add a note under `Code actions` to the ReleaseNotes
(clang-tools-extra/docs/ReleaseNotes.rst) about your change.
Looks like a good tweak to have, looking forward to it
https://github.com/llvm/llvm-project/pull/69693
@@ -0,0 +1,262 @@
+//===--- InlineConceptRequirement.cpp *-
C++-*-===//
+//
+// 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: Ap
@@ -0,0 +1,262 @@
+//===--- InlineConceptRequirement.cpp *-
C++-*-===//
+//
+// 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: Ap
@@ -0,0 +1,262 @@
+//===--- InlineConceptRequirement.cpp *-
C++-*-===//
+//
+// 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: Ap
@@ -0,0 +1,262 @@
+//===--- InlineConceptRequirement.cpp *-
C++-*-===//
+//
+// 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: Ap
5chmidti wrote:
Clangd (actually large parts of llvm) don't use trailing return types, you will
probably have to change those. This is a bit of a chore, so you could wait for
another reviewer to confirm that for you.
https://github.com/llvm/llvm-project/pull/
https://github.com/5chmidti edited
https://github.com/llvm/llvm-project/pull/69693
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Venyla edited https://github.com/llvm/llvm-project/pull/69693
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Venyla updated
https://github.com/llvm/llvm-project/pull/69693
>From 1a49159a4d8186223c0cf87e9225aafe0fd6a2dd Mon Sep 17 00:00:00 2001
From: Vina Zahnd
Date: Fri, 20 Oct 2023 10:01:54 +0200
Subject: [PATCH] [clangd] Add tweak to inline concept requirements
Co-authored-by: Je
https://github.com/Venyla updated
https://github.com/llvm/llvm-project/pull/69693
>From 2b1f56a758d397649c94717f4a030c04a532bde7 Mon Sep 17 00:00:00 2001
From: Vina Zahnd
Date: Fri, 20 Oct 2023 10:01:54 +0200
Subject: [PATCH] [clangd] Add tweak to inline concept requirements
Co-authored-by: Je
https://github.com/Venyla updated
https://github.com/llvm/llvm-project/pull/69693
>From 9e2a80fd1a7ae61ce71ba0caabd2c6a88c662889 Mon Sep 17 00:00:00 2001
From: Vina Zahnd
Date: Fri, 20 Oct 2023 10:01:54 +0200
Subject: [PATCH] [clangd] Add tweak to inline concept requirements
Co-authored-by: Je
https://github.com/Venyla created
https://github.com/llvm/llvm-project/pull/69693
[Screencast from 2023-10-16
15-34-29.webm](https://github.com/sa-concept-refactoring/llvm-project/assets/7629727/bd2c8790-c40f-4834-8523-6d16fca0d498)
https://github.com/llvm/llvm-project/assets/6904387/95fa5ab1-
15 matches
Mail list logo
