[clang] [clang] Allow parentheses around CTAD declarators (PR #132829)

2025-04-02 Thread via cfe-commits
https://github.com/offsetof updated https://github.com/llvm/llvm-project/pull/132829 >From 66833f41d26a6c6355ef67b2f9041ba771b690b7 Mon Sep 17 00:00:00 2001 From: offsetof Date: Mon, 24 Mar 2025 20:51:23 + Subject: [PATCH 1/3] [clang] Allow parentheses around CTAD declarators --- clang/do

[clang] [clang] Allow parentheses around CTAD declarators (PR #132829)

2025-03-25 Thread via cfe-commits
https://github.com/offsetof updated https://github.com/llvm/llvm-project/pull/132829 >From 66833f41d26a6c6355ef67b2f9041ba771b690b7 Mon Sep 17 00:00:00 2001 From: offsetof Date: Mon, 24 Mar 2025 20:51:23 + Subject: [PATCH 1/2] [clang] Allow parentheses around CTAD declarators --- clang/do

[clang] [clang] Allow parentheses around CTAD declarators (PR #132829)

2025-03-25 Thread via cfe-commits
cor3ntin wrote: > CC @Endilll Should this also add a test to > [cwg23xx.cpp](https://github.com/llvm/llvm-project/blob/main/clang/test/CXX/drs/cwg23xx.cpp)? Yes, please! https://github.com/llvm/llvm-project/pull/132829 ___ cfe-commits mailing list cf

[clang] [clang] Allow parentheses around CTAD declarators (PR #132829)

2025-03-25 Thread via cfe-commits
https://github.com/Sirraide edited https://github.com/llvm/llvm-project/pull/132829 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Allow parentheses around CTAD declarators (PR #132829)

2025-03-25 Thread via cfe-commits
Sirraide wrote: CC @Endilll Should this also add a test to [cwg23xx.cpp](https://github.com/llvm/llvm-project/blob/main/clang/test/CXX/drs/cwg23xx.cpp)? https://github.com/llvm/llvm-project/pull/132829 ___ cfe-commits mailing list cfe-commits@lists.

[clang] [clang] Allow parentheses around CTAD declarators (PR #132829)

2025-03-25 Thread via cfe-commits
https://github.com/Sirraide approved this pull request. Actually, I just checked #39811, and apparently this was a CWG that has been resolved, so in that case this seems fine. https://github.com/llvm/llvm-project/pull/132829 ___ cfe-commits mailing li

[clang] [clang] Allow parentheses around CTAD declarators (PR #132829)

2025-03-25 Thread via cfe-commits
https://github.com/Sirraide commented: So the implementation of this seems fine, but looking at how other compilers handle this, GCC and MSVC accept this, whereas EDG rejects. I agree w/ the comment that that is a bit silly to reject this, but can you check what the standard has to say about t

[clang] [clang] Allow parentheses around CTAD declarators (PR #132829)

2025-03-24 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: None (offsetof) Changes Fixes #39811 --- Full diff: https://github.com/llvm/llvm-project/pull/132829.diff 5 Files Affected: - (modified) clang/docs/ReleaseNotes.rst (+2) - (modified) clang/include/clang/Basic/DiagnosticSemaKinds.td (+3

[clang] [clang] Allow parentheses around CTAD declarators (PR #132829)

2025-03-24 Thread via cfe-commits
https://github.com/offsetof created https://github.com/llvm/llvm-project/pull/132829 Fixes #39811 >From 66833f41d26a6c6355ef67b2f9041ba771b690b7 Mon Sep 17 00:00:00 2001 From: offsetof Date: Mon, 24 Mar 2025 20:51:23 + Subject: [PATCH] [clang] Allow parentheses around CTAD declarators ---