@@ -19,4 +19,8 @@ void f2(void) {
struct U u;
}
-
+void f3(void) {
shafik wrote:
So unlike the original test case this one does not seem to cause a crash. Can
we add a test case for that as well to verify we are fixing that issue as well.
cor3ntin wrote:
The change looks good to me, but it is missing a release note in
clang/docs/ReleaseNotes.rst
https://github.com/llvm/llvm-project/pull/69432
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Qiu Chaofan (ecnelises)
Changes
In non C++ mode, struct definitions does not create a scope for declaration.
Fixes #41302
Fixes #44080
---
Full diff: https://github.com/llvm/llvm-project/pull/69432.diff
3 Files Affected:
-
https://github.com/ecnelises updated
https://github.com/llvm/llvm-project/pull/69432
>From f97807afc177ab04c97d0c346081fc92d3e79e6c Mon Sep 17 00:00:00 2001
From: Qiu Chaofan
Date: Wed, 18 Oct 2023 16:02:02 +0800
Subject: [PATCH] [Clang][Sema] Skip RecordDecl when checking scope of
https://github.com/ecnelises created
https://github.com/llvm/llvm-project/pull/69432
In non C++ mode, struct definitions does not create a scope for declaration.
Fixes #41302
Fixes #44080
>From 6828025e788347f21a41d4a9aa138af032017b80 Mon Sep 17 00:00:00 2001
From: Qiu Chaofan
Date: Wed,