* [PATCH] PR c++/103338 - Add testcase for issue fixed by recent commit [not found] <20240604155436.1255-1-simon@nasilyan.com> @ 2024-06-04 15:54 ` Simon Martin 2024-06-04 16:12 ` Jason Merrill 0 siblings, 1 reply; 3+ messages in thread From: Simon Martin @ 2024-06-04 15:54 UTC (permalink / raw) To: gcc-patches The case in that PR used to ICE until commit f04dc89. This patch simply adds the case to the testsuite. Successfully tested on x86_64-pc-linux-gnu. PR c++/1033388 gcc/testsuite/ChangeLog: * g++.dg/parse/crash73.C: New test. --- gcc/testsuite/g++.dg/parse/crash73.C | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) create mode 100644 gcc/testsuite/g++.dg/parse/crash73.C diff --git a/gcc/testsuite/g++.dg/parse/crash73.C b/gcc/testsuite/g++.dg/parse/crash73.C new file mode 100644 index 00000000000..5923b98b719 --- /dev/null +++ b/gcc/testsuite/g++.dg/parse/crash73.C @@ -0,0 +1,19 @@ +// PR c++/1033388 +// { dg-do compile { target c++11 } } + +template<class...> +struct zip_view { + struct Iterator; +}; + +template<class...> +struct zip_transform_view; + +template<class... Views> +struct zip_view<Views...>::Iterator { // { dg-error "no class template" } + template<class... Uiews> + template<bool> + friend class zip_transform_view<Uiews...>::Iterator; +}; + +zip_view<>::Iterator iter; -- 2.44.0 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] PR c++/103338 - Add testcase for issue fixed by recent commit 2024-06-04 15:54 ` [PATCH] PR c++/103338 - Add testcase for issue fixed by recent commit Simon Martin @ 2024-06-04 16:12 ` Jason Merrill [not found] ` <C4F9A382-D952-4703-AAFC-44D1E7980BD2@nasilyan.com> 0 siblings, 1 reply; 3+ messages in thread From: Jason Merrill @ 2024-06-04 16:12 UTC (permalink / raw) To: Simon Martin, gcc-patches; +Cc: Patrick Palka On 6/4/24 11:54, Simon Martin wrote: > The case in that PR used to ICE until commit f04dc89. Interesting, I don't remember expecting that patch to change behavior at all. BTW, it looks like your recent commits and emails have had non-conventional subject lines; see https://gcc.gnu.org/contribute.html#patches for more guidance. For instance, the subject for this patch could be c++: add testcase for PR103338 OK with that adjustment. > This patch simply adds > the case to the testsuite. > > Successfully tested on x86_64-pc-linux-gnu. > > PR c++/1033388 > > gcc/testsuite/ChangeLog: > > * g++.dg/parse/crash73.C: New test. > > --- > gcc/testsuite/g++.dg/parse/crash73.C | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > create mode 100644 gcc/testsuite/g++.dg/parse/crash73.C > > diff --git a/gcc/testsuite/g++.dg/parse/crash73.C b/gcc/testsuite/g++.dg/parse/crash73.C > new file mode 100644 > index 00000000000..5923b98b719 > --- /dev/null > +++ b/gcc/testsuite/g++.dg/parse/crash73.C > @@ -0,0 +1,19 @@ > +// PR c++/1033388 > +// { dg-do compile { target c++11 } } > + > +template<class...> > +struct zip_view { > + struct Iterator; > +}; > + > +template<class...> > +struct zip_transform_view; > + > +template<class... Views> > +struct zip_view<Views...>::Iterator { // { dg-error "no class template" } > + template<class... Uiews> > + template<bool> > + friend class zip_transform_view<Uiews...>::Iterator; > +}; > + > +zip_view<>::Iterator iter; ^ permalink raw reply [flat|nested] 3+ messages in thread
[parent not found: <C4F9A382-D952-4703-AAFC-44D1E7980BD2@nasilyan.com>]
* Re: [PATCH] PR c++/103338 - Add testcase for issue fixed by recent commit [not found] ` <C4F9A382-D952-4703-AAFC-44D1E7980BD2@nasilyan.com> @ 2024-06-04 16:42 ` Simon Martin 0 siblings, 0 replies; 3+ messages in thread From: Simon Martin @ 2024-06-04 16:42 UTC (permalink / raw) To: Jason Merrill; +Cc: gcc-patches, Patrick Palka Hi Jason, On 4 Jun 2024, at 18:12, Jason Merrill wrote: > On 6/4/24 11:54, Simon Martin wrote: >> The case in that PR used to ICE until commit f04dc89. > > Interesting, I don't remember expecting that patch to change behavior > at all. This is the patch that git bisect identified. I have to admit that I did not look further. > BTW, it looks like your recent commits and emails have had > non-conventional subject lines; see > https://gcc.gnu.org/contribute.html#patches for more guidance. > Thanks for the pointer and apologies for not providing great subject lines; I’ll fix this moving forward (starting with that patch). > For instance, the subject for this patch could be > > c++: add testcase for PR103338 > > OK with that adjustment. Thanks again. > >> This patch simply adds >> the case to the testsuite. >> >> Successfully tested on x86_64-pc-linux-gnu. >> >> PR c++/1033388 >> >> gcc/testsuite/ChangeLog: >> >> * g++.dg/parse/crash73.C: New test. >> >> --- >> gcc/testsuite/g++.dg/parse/crash73.C | 19 +++++++++++++++++++ >> 1 file changed, 19 insertions(+) >> create mode 100644 gcc/testsuite/g++.dg/parse/crash73.C >> >> diff --git a/gcc/testsuite/g++.dg/parse/crash73.C >> b/gcc/testsuite/g++.dg/parse/crash73.C >> new file mode 100644 >> index 00000000000..5923b98b719 >> --- /dev/null >> +++ b/gcc/testsuite/g++.dg/parse/crash73.C >> @@ -0,0 +1,19 @@ >> +// PR c++/1033388 >> +// { dg-do compile { target c++11 } } >> + >> +template<class...> >> +struct zip_view { >> + struct Iterator; >> +}; >> + >> +template<class...> >> +struct zip_transform_view; >> + >> +template<class... Views> >> +struct zip_view<Views...>::Iterator { // { dg-error "no class >> template" } >> + template<class... Uiews> >> + template<bool> >> + friend class zip_transform_view<Uiews...>::Iterator; >> +}; >> + >> +zip_view<>::Iterator iter; ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-06-04 16:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20240604155436.1255-1-simon@nasilyan.com>
2024-06-04 15:54 ` [PATCH] PR c++/103338 - Add testcase for issue fixed by recent commit Simon Martin
2024-06-04 16:12 ` Jason Merrill
[not found] ` <C4F9A382-D952-4703-AAFC-44D1E7980BD2@nasilyan.com>
2024-06-04 16:42 ` Simon Martin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).