https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/92672
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -989,6 +991,11 @@ class Sema final : public SemaBase {
return *CodeCompletionPtr;
}
+ SemaConcept &Concept() {
cor3ntin wrote:
```suggestion
SemaConcept &Concepts() {
```
https://github.com/llvm/llvm-project/pull/92672
___
https://github.com/cor3ntin requested changes to this pull request.
It makes very little sense to split templates from concepts imo.
(so we run into the risk of causing churn twice)
https://github.com/llvm/llvm-project/pull/92672
___
cfe-commits mailin
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/92672
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 58c778565cd64a69ea86e7e67e6a87fff3b0b224
354e8ff9f82d26b3611766d6b78b5da76ec87e69 --
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Vlad Serebrennikov (Endilll)
Changes
This patch moves `Sema` functions that handle C++20 concepts into the new
`SemaConcept` class. This continues previous efforts to split `Sema` up.
Additional context can be found in #84184.
This patch
