https://github.com/shafik commented:
LGTM
https://github.com/llvm/llvm-project/pull/78040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Endilll closed
https://github.com/llvm/llvm-project/pull/78040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/78040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Endilll updated
https://github.com/llvm/llvm-project/pull/78040
>From 62620337b64c64535d76c5003f9acd450ab527f7 Mon Sep 17 00:00:00 2001
From: Vlad Serebrennikov
Date: Sat, 13 Jan 2024 17:32:37 +0300
Subject: [PATCH 1/2] [clang] Add test for CWG1350
Test is based on
cor3ntin wrote:
For completeness, can you add a couple of tests covering the note (ie, template
case?)
Otherwise, LGTM
https://github.com/llvm/llvm-project/pull/78040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Endilll wrote:
I was way too deep into P0136R1 [Rewording inheriting constructors (core issue
1941 et al)](https://wg21.link/p0136r1) when I realized Richard has written
tests for all issues covered there long ago. So I decided to carry on and check
whether those tests would benefit from
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Vlad Serebrennikov (Endilll)
Changes
Test is based on [P0136R1](https://wg21.link/p0136r1) wording instead of
proposed resolution in the issue itself.
This patch also expands related CWG1573 test with an additional test case.
Existing
https://github.com/Endilll created
https://github.com/llvm/llvm-project/pull/78040
Test is based on [P0136R1](https://wg21.link/p0136r1) wording instead of
proposed resolution in the issue itself.
This patch also expands related CWG1573 test with an additional test case.
Existing `3.9`