https://github.com/zwuis created
https://github.com/llvm/llvm-project/pull/151995
Make it easier for us to add abi versions.
Close #144332
>From 8881838e71226a31f06333ef55a73718f32b Mon Sep 17 00:00:00 2001
From: Yanzuo Liu
Date: Tue, 5 Aug 2025 00:19:53 +0800
Subject: [PATCH] Enumerate C
zwuis wrote:
> Hi! I want to close this PR. What do I need to do?
You can just close this PR.
https://github.com/llvm/llvm-project/pull/144828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-
@@ -135,7 +135,7 @@ int32_t ByteCodeEmitter::getOffset(LabelTy Label) {
/// Helper to write bytecode and bail out if 32-bit offsets become invalid.
/// Pointers will be automatically marshalled as 32-bit IDs.
template
-static void emit(Program &P, std::vector &Code, const T &V
https://github.com/zwuis commented:
LGTM but please let other people take a look.
https://github.com/llvm/llvm-project/pull/151790
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,13 @@
+// RUN: %clang_cc1 -verify -std=c++17 -pedantic-errors %s
+// RUN: cp %s %t
+// RUN: not %clang_cc1 -x c++ -std=c++17 -fixit %t
+// RUN: %clang_cc1 -Wall -pedantic-errors -x c++ -std=c++17 %t
+
+/* This is a test of the various code modification hints that only
+
https://github.com/zwuis closed https://github.com/llvm/llvm-project/pull/150814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
> Is there no issue associated with the crash?
I ran into this assertion failure when trying to implement cwg712. I tried to
search issues but found nothing.
https://github.com/llvm/llvm-project/pull/150814
___
cfe-commits mailing list
c
zwuis wrote:
> Then what is the benefits?
Adding `__builtin_lambda_this()` is prerequisite of fixing the issue which this
PR tries to fix, because we cannot access the caller of lambda `operator()`
without adding compiler magic such as builtin functions.
> Whatever it is, I feel this can/shou
zwuis wrote:
> Would you like to give a brief introduction for your design. I didn't figure
> it out.
- The function signature of `__builtin_lambda_this()` is `void*()`.
- Sema part: if it is a non-staic lambda expression (similar with this PR),
create an expression `static_cast(__builtin_lamb
zwuis wrote:
Hi. I tried to fix this issue locally by adding a function
`__builtin_lambda_this()` and it seems to work. I can create PR if you think
this is a good approach. And I have some questions:
1. Should this builtin function handle lambda expression only? What about
`__builtin_member_
https://github.com/zwuis created
https://github.com/llvm/llvm-project/pull/150814
Add missing `ActOnFinishFullExpr` to `BuildCXXAssumeExpr`. We did it during
template instantiation but forgot non-template case.
>From 28d6be07b613662c5a0d9491c67cf880eb42eadc Mon Sep 17 00:00:00 2001
From: Yanzu
zwuis wrote:
I added comment `// FIXME (GH147000): duplicate diagnostics` to
`clang/test/SemaCXX/nested-name-spec.cpp` in #147003. Please remove the comment
if that issue is fixed by this patch.
https://github.com/llvm/llvm-project/pull/147835
___
cf
zwuis wrote:
Can you manually backport this patch? See #150131.
https://github.com/llvm/llvm-project/pull/149227
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
I think we need to test partial specialization as well.
https://github.com/llvm/llvm-project/pull/150003
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/150003
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -44,3 +44,18 @@ void testTemplates() {
throwErrorTemplate("ERROR");
(void)ensureZeroTemplate(42);
}
+
+// Ensure that explicit specialization of a member function does not inherit
+// the warning from the primary template.
+
+template
+struct S {
+ void f();
+};
+
+temp
zwuis wrote:
Can you add this test case?
```cpp
template
struct S {
void f();
};
template
void S::f() {}
template<>
void S::f() { throw 0; }
```
https://github.com/llvm/llvm-project/pull/150003
___
@@ -547,6 +547,15 @@ static void DoEmitAvailabilityWarning(Sema &S,
AvailabilityResult K,
return;
}
case AR_Deprecated:
+// Suppress -Wdeprecated-declarations in purely implicit special-member
functions.
+if (auto *MD = dyn_cast_if_present(S.getCurFunctionDecl
@@ -547,6 +547,15 @@ static void DoEmitAvailabilityWarning(Sema &S,
AvailabilityResult K,
return;
}
case AR_Deprecated:
+// Suppress -Wdeprecated-declarations in purely implicit special-member
functions.
+if (auto *MD = dyn_cast_if_present(S.getCurFunctionDecl
@@ -181,14 +181,14 @@ template class
DynamicRecursiveASTVisitorBase {
///
/// \returns false if the visitation was terminated early, true
/// otherwise (including when the argument is a Null type).
- virtual bool TraverseType(QualType T);
+ virtual bool TraverseType(Q
https://github.com/zwuis closed https://github.com/llvm/llvm-project/pull/149123
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -575,7 +575,8 @@ bool CheckConst(InterpState &S, CodePtr OpPC, const Pointer
&Ptr) {
// The This pointer is writable in constructors and destructors,
// even if isConst() returns true.
- if (llvm::find(S.InitializingBlocks, Ptr.block()))
+ if (llvm::find(S.Initializi
@@ -0,0 +1,155 @@
+=
+C++ Type Aware Allocators
+=
+
+.. contents::
+ :local:
+
+Introduction
+
+
+Clang includes an implementation of P2719 "Type-aware allocation and
deallocation
+functions".
+
+This is a feature tha
https://github.com/zwuis approved this pull request.
Thanks!
https://github.com/llvm/llvm-project/pull/148576
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/148576
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
Please update `clang/docs/LanguageExtensions.rst`.
https://github.com/llvm/llvm-project/pull/148576
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis closed https://github.com/llvm/llvm-project/pull/148195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
Thanks for your review.
https://github.com/llvm/llvm-project/pull/148195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
> > I'm not sure if I should update
> > `llvm/include/llvm/Testing/Demangle/DemangleTestCases.inc` and
> > `libcxxabi/test/DemangleTestCases.inc`.
>
> Why would you need to? There are no changes to mangling/demangling here AFAICT
These files test demangling `RecordDecl::isInjecte
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/148195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
I'm not sure if I should update
`llvm/include/llvm/Testing/Demangle/DemangleTestCases.inc` and
`libcxxabi/test/DemangleTestCases.inc`.
https://github.com/llvm/llvm-project/pull/148195
___
cfe-commits mailing list
cfe-commits@lists.llvm.o
https://github.com/zwuis created
https://github.com/llvm/llvm-project/pull/148195
Co-authored-by: Matheus Izvekov
>From 306049aa7dd17f6683935d631b3ad222b268a3f2 Mon Sep 17 00:00:00 2001
From: Yanzuo Liu
Date: Fri, 11 Jul 2025 18:17:05 +0800
Subject: [PATCH] (`RecordDecl` -> `CXXRecordDecl`)`:
zwuis wrote:
Do you need the help to merge this PR?
https://github.com/llvm/llvm-project/pull/139348
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -547,6 +547,20 @@ static void DoEmitAvailabilityWarning(Sema &S,
AvailabilityResult K,
return;
}
case AR_Deprecated:
+// Suppress -Wdeprecated-declarations in purely implicit special-member
functions.
+if (const FunctionDecl *FD = S.getCurFunctionDecl()) {
@@ -547,6 +547,20 @@ static void DoEmitAvailabilityWarning(Sema &S,
AvailabilityResult K,
return;
}
case AR_Deprecated:
+// Suppress -Wdeprecated-declarations in purely implicit special-member
functions.
+if (const FunctionDecl *FD = S.getCurFunctionDecl()) {
https://github.com/zwuis closed https://github.com/llvm/llvm-project/pull/147003
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis updated
https://github.com/llvm/llvm-project/pull/147003
>From 32e2c12d44da29e20212bdeed3628d4c736191ad Mon Sep 17 00:00:00 2001
From: Yanzuo Liu
Date: Fri, 4 Jul 2025 13:58:17 +0800
Subject: [PATCH] Do not perform error recovery for invalid member
using-declaration in
zwuis wrote:
Can we merge this with private email address?
https://github.com/llvm/llvm-project/pull/146433
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis created
https://github.com/llvm/llvm-project/pull/147003
Previously, Clang tried to perform error recovery for invalid member
using-declaration whose nested-name-specifier refers to its own class in C++20+
mode, which causes crash.
```cpp
template struct V {};
struct
@@ -6,9 +6,21 @@ int main() {
A a {};
a.
}
-// RUN: %clang_cc1 -cc1 -fsyntax-only -code-completion-at=%s:%(line-2):5
-std=c++23 %s | FileCheck %s
-// CHECK: COMPLETION: A : A::
-// CHECK-NEXT: COMPLETION: foo : [#void#]foo(<#int arg#>)
-// CHECK-NEXT: COMPLETION: operator=
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/146649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis commented:
Thank you for the fix!
https://github.com/llvm/llvm-project/pull/146649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis closed https://github.com/llvm/llvm-project/pull/146474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
Thank you for your review.
https://github.com/llvm/llvm-project/pull/146474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -17,7 +18,7 @@ struct fake_tuple {
int arr[4] = {1, 2, 3, 6};
template
- int get() {
+ constexpr int& get() {
zwuis wrote:
> Were these two functions never called in a constant context before?
Yes. Only semantic analysis was performed on these func
zwuis wrote:
Please update "clang/www/cxx_status.html".
https://github.com/llvm/llvm-project/pull/146461
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis closed https://github.com/llvm/llvm-project/pull/143492
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
Thank you for your review!
Failed test in CI is not related.
https://github.com/llvm/llvm-project/pull/143492
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis updated
https://github.com/llvm/llvm-project/pull/143492
>From 658f3d3cb72ff9fbdb2315f894f39956eeabf135 Mon Sep 17 00:00:00 2001
From: Yanzuo Liu
Date: Tue, 10 Jun 2025 16:41:57 +0800
Subject: [PATCH 1/5] Apply CWG400 'Using-declarations and the "struct hack"'
to C++98
https://github.com/zwuis updated
https://github.com/llvm/llvm-project/pull/143492
>From 658f3d3cb72ff9fbdb2315f894f39956eeabf135 Mon Sep 17 00:00:00 2001
From: Yanzuo Liu
Date: Tue, 10 Jun 2025 16:41:57 +0800
Subject: [PATCH 1/4] Apply CWG400 'Using-declarations and the "struct hack"'
to C++98
zwuis wrote:
What about "ExplicitObjectMemberFunction"? It seems that this part of feature
has not been tested.
https://github.com/llvm/llvm-project/pull/146258
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
https://github.com/zwuis approved this pull request.
https://github.com/llvm/llvm-project/pull/146282
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -301,177 +735,284 @@ optimized to the equivalent of:
std::cout << a+5 << "\n";
}
-It should now be obvious why the value of `__int_32_0` remains unchanged
-throughout the function. It is important to recognize that `__int_32_0`
-does not directly correspond to `a`, bu
@@ -301,177 +735,284 @@ optimized to the equivalent of:
std::cout << a+5 << "\n";
}
-It should now be obvious why the value of `__int_32_0` remains unchanged
-throughout the function. It is important to recognize that `__int_32_0`
-does not directly correspond to `a`, bu
zwuis wrote:
reStructuredText (`.rst`) doesn't support Markdown style link (`[text](link)`).
Can you fix it?
https://github.com/llvm/llvm-project/pull/142651
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
zwuis wrote:
Thank you for the patch! Please add a release note to
"clang/docs/ReleaseNotes.rst" so users can know the improvement. Please add a
tag to the beginning of the PR title in square brackets like other PRs. E.g.
`[clang]`.
https://github.com/llvm/llvm-project/pull/146103
___
zwuis wrote:
> > I think we need an entry in "Potentially breaking changes" section.
>
> This is still missing.
I think [this
commit](https://github.com/llvm/llvm-project/commit/c9e11f8d3d2c277fca8ec986b0c01ac71e0e4161)
move the release note to the right place. Can you check this again?
http
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/145784
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis commented:
Please add a tag to the beginning of the PR title in square brackets like other
PRs. E.g. `[clang-tidy]`.
https://github.com/llvm/llvm-project/pull/145784
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/143492
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -101,11 +101,10 @@ class Token {
/// "if (Tok.is(tok::l_brace)) {...}".
bool is(tok::TokenKind K) const { return Kind == K; }
bool isNot(tok::TokenKind K) const { return Kind != K; }
- bool isOneOf(tok::TokenKind K1, tok::TokenKind K2) const {
-return is(K1) || is
@@ -0,0 +1,79 @@
+// RUN: %clangxx -std=c++23 -fsyntax-only -Xclang -verify %s
zwuis wrote:
Use `%clang_cc1` for frontend tests.
`-Xclang` can be removed.
https://github.com/llvm/llvm-project/pull/145164
___
cfe-commi
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/145164
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,79 @@
+// RUN: %clangxx -std=c++23 -fsyntax-only -Xclang -verify %s
+
+#include
+#include
+#include
+#include
+#include
+#include
+#include
+#include
zwuis wrote:
Do not include files outside test folders. You can search `namespace std` to
see
@@ -57,6 +57,31 @@ enum LifetimeKind {
};
using LifetimeResult =
llvm::PointerIntPair;
+
+/// Returns true if the given entity is part of a range-based for loop and
+/// should trigger lifetime extension under C++23 rules.
+///
+/// This handles both explicit range loop var
https://github.com/zwuis commented:
Thank you for the patch!
Please add a release note in `clang/docs/ReleaseNotes.rst` so that users can
know the improvement.
https://github.com/llvm/llvm-project/pull/145164
___
cfe-commits mailing list
cfe-commits@
@@ -681,6 +705,30 @@ static void CheckFallThroughForBody(Sema &S, const Decl
*D, const Stmt *Body,
if (CD.diag_FallThrough_HasNoReturn)
S.Diag(RBrace, CD.diag_FallThrough_HasNoReturn) << CD.FunKind;
} else if (!ReturnsVoid && CD.diag_FallThrough_ReturnsNonVoi
https://github.com/zwuis commented:
Thank you for the patch! Please add a release note in
`clang/docs/ReleaseNotes.rst`.
I'm not sure if this case should be handled:
```cpp
if (condition) {
throw 1;
} else {
throw 2;
}
```
Maybe we need to open an issue for it.
https://github.com/llvm/llvm
@@ -624,8 +624,32 @@ struct CheckFallThroughDiagnostics {
}
};
-} // anonymous namespace
+bool isKnownToAlwaysThrow(const FunctionDecl *FD) {
+ if (!FD->hasBody())
+return false;
+ const Stmt *Body = FD->getBody();
+ const Stmt *OnlyStmt = nullptr;
+
+ if (const auto
@@ -681,6 +705,30 @@ static void CheckFallThroughForBody(Sema &S, const Decl
*D, const Stmt *Body,
if (CD.diag_FallThrough_HasNoReturn)
S.Diag(RBrace, CD.diag_FallThrough_HasNoReturn) << CD.FunKind;
} else if (!ReturnsVoid && CD.diag_FallThrough_ReturnsNonVoi
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/145166
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
You should fork https://github.com/Ritanya-B-Bharadwaj/llvm-project and create
Pull Request there.
https://github.com/llvm/llvm-project/pull/145074
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
zwuis wrote:
You should fork https://github.com/Ritanya-B-Bharadwaj/llvm-project and create
Pull Request there.
https://github.com/llvm/llvm-project/pull/144622
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
zwuis wrote:
You should fork https://github.com/Ritanya-B-Bharadwaj/llvm-project and create
Pull Request there.
https://github.com/llvm/llvm-project/pull/144935
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
zwuis wrote:
You should fork https://github.com/Ritanya-B-Bharadwaj/llvm-project and create
Pull Request there.
https://github.com/llvm/llvm-project/pull/145042
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
zwuis wrote:
You should fork https://github.com/Ritanya-B-Bharadwaj/llvm-project and create
Pull Request there.
https://github.com/llvm/llvm-project/pull/144478
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
zwuis wrote:
Thank you for the patch! Please add tests in `clang/test/` to make sure this
fix actually works. Please add a release note in `clang/docs/ReleaseNotes.rst`
so that users can know the improvement.
https://github.com/llvm/llvm-project/pull/144828
zwuis wrote:
Where is the fix? Only test file is modified currently.
https://github.com/llvm/llvm-project/pull/144956
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
Copying a comment from #144622:
> Please start a thread on https://discourse.llvm.org/ describing the problem
> you're trying to solve, and your proposed solution.
https://github.com/llvm/llvm-project/pull/144478
___
cfe-commits mailing
zwuis wrote:
Is it necessary to have this feature? Clangd already [supports
it](https://clangd.llvm.org/config.html#inlayhints).
https://github.com/llvm/llvm-project/pull/144478
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
https://github.com/zwuis updated
https://github.com/llvm/llvm-project/pull/143492
>From 658f3d3cb72ff9fbdb2315f894f39956eeabf135 Mon Sep 17 00:00:00 2001
From: Yanzuo Liu
Date: Tue, 10 Jun 2025 16:41:57 +0800
Subject: [PATCH 1/2] Apply CWG400 'Using-declarations and the "struct hack"'
to C++98
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/143492
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/143492
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
Please add a release note.
https://github.com/llvm/llvm-project/pull/144286
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -905,7 +905,7 @@ C++20 implementation status
https://wg21.link/P2788R0";>P2788R0 (DR)
-No
+Clang 17
zwuis wrote:
```suggestion
Clang 17
```
https://github.com/llvm/llvm-project/pull/144214
___
@@ -2276,6 +2277,134 @@ static void DiagnoseNonTriviallyCopyableReason(Sema
&SemaRef,
SemaRef.Diag(D->getLocation(), diag::note_defined_here) << D;
}
+static bool hasMixedAccessSpecifier(const CXXRecordDecl *D) {
+ AccessSpecifier FirstAccess = AS_none;
+ for (const Field
zwuis wrote:
I wrote wrong test case in PR description. I apologize. The following is being
fixed:
```cpp
struct A {};
struct B : A {
using B::A;
};
```
PR description was updated. I'm away from my computer and I will add this code
to test files next Monday.
https://github.com/llvm/llvm-pr
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/143492
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/143492
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,28 @@
+// RUN: %clang_cc1 -fsyntax-only -verify -Wunterminated-string-initialization
%s -x c
+// RUN: %clang_cc1 -fsyntax-only -verify -Wunterminated-string-initialization
%s -x c++
+
+
+// In C, the following examples are fine:
+#if __cplusplus
zwuis
zwuis wrote:
I'm not sure if I should modify CWG400 implementation status to "Clang 21". CC
@Endilll
https://github.com/llvm/llvm-project/pull/143492
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
https://github.com/zwuis created
https://github.com/llvm/llvm-project/pull/143492
It seems that we applied CWG400 to C++11 and newer version. But CWG400 should
also be applied to C++98.
>From 658f3d3cb72ff9fbdb2315f894f39956eeabf135 Mon Sep 17 00:00:00 2001
From: Yanzuo Liu
Date: Tue, 10 Jun
zwuis wrote:
Ping.
https://github.com/llvm/llvm-project/pull/131054
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
IIUC the `FIXME` comment means that the time complexity of this part of code is
not good enough and we can make it faster.
https://github.com/llvm/llvm-project/pull/143275
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
zwuis wrote:
Yes. I still don't have commit access. Thank you!
https://github.com/llvm/llvm-project/pull/142906
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -745,6 +755,16 @@ class alignas(void *) Stmt {
unsigned TemplateDepth;
};
+ class ChooseExprBitfields {
+friend class ASTStmtReader;
+friend class ChooseExpr;
+
+LLVM_PREFERRED_TYPE(ExprBitfields)
+unsigned : NumExprBits;
+
+bool CondIsTrue : 1;
-
zwuis wrote:
Can you handle this case `[[using gnu : deprected]]`?
https://github.com/llvm/llvm-project/pull/141305
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -7639,6 +7639,8 @@ def warn_param_mismatched_alignment : Warning<
def err_objc_object_assignment : Error<
"cannot assign to class object (%0 invalid)">;
+def err_typecheck_array_prvalue_operand : Error<
+ "array prvalue is not permitted">;
zwuis wrote:
@@ -7639,6 +7639,8 @@ def warn_param_mismatched_alignment : Warning<
def err_objc_object_assignment : Error<
"cannot assign to class object (%0 invalid)">;
+def err_typecheck_array_prvalue_operand : Error<
+ "array prvalue is not permitted">;
zwuis wrote:
zwuis wrote:
This is related to CWG2548. CC @Endilll
https://github.com/llvm/llvm-project/pull/140702
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 250 matches
Mail list logo
