Re: [PATCH] c++: ICE with TTP [PR96097]

2024-09-11 Thread Jason Merrill
On 9/11/24 10:53 AM, Patrick Palka wrote: On Wed, 11 Sep 2024, Patrick Palka wrote: On Wed, 11 Sep 2024, Patrick Palka wrote: On Wed, 4 Sep 2024, Marek Polacek wrote: On Wed, Sep 04, 2024 at 10:58:25AM -0400, Jason Merrill wrote: On 9/3/24 6:12 PM, Marek Polacek wrote: Bootstrapped/regtes

Re: [PATCH] c++: ICE with TTP [PR96097]

2024-09-11 Thread Patrick Palka
On Wed, 11 Sep 2024, Patrick Palka wrote: > On Wed, 11 Sep 2024, Patrick Palka wrote: > > > On Wed, 4 Sep 2024, Marek Polacek wrote: > > > > > On Wed, Sep 04, 2024 at 10:58:25AM -0400, Jason Merrill wrote: > > > > On 9/3/24 6:12 PM, Marek Polacek wrote: > > > > > Bootstrapped/regtested on x86_64

Re: [PATCH] c++: ICE with TTP [PR96097]

2024-09-11 Thread Patrick Palka
On Wed, 11 Sep 2024, Patrick Palka wrote: > On Wed, 4 Sep 2024, Marek Polacek wrote: > > > On Wed, Sep 04, 2024 at 10:58:25AM -0400, Jason Merrill wrote: > > > On 9/3/24 6:12 PM, Marek Polacek wrote: > > > > Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for trunk/14? > > > > > > The change t

Re: [PATCH] c++: ICE with TTP [PR96097]

2024-09-11 Thread Patrick Palka
On Wed, 4 Sep 2024, Marek Polacek wrote: > On Wed, Sep 04, 2024 at 10:58:25AM -0400, Jason Merrill wrote: > > On 9/3/24 6:12 PM, Marek Polacek wrote: > > > Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for trunk/14? > > > > The change to return bool seems like unrelated cleanup; please push t

Re: [PATCH] c++: ICE with TTP [PR96097]

2024-09-04 Thread Marek Polacek
On Wed, Sep 04, 2024 at 10:58:25AM -0400, Jason Merrill wrote: > On 9/3/24 6:12 PM, Marek Polacek wrote: > > Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for trunk/14? > > The change to return bool seems like unrelated cleanup; please push that > separately on trunk only. Done. > > + /

Re: [PATCH] c++: ICE with TTP [PR96097]

2024-09-04 Thread Jason Merrill
On 9/3/24 6:12 PM, Marek Polacek wrote: Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for trunk/14? The change to return bool seems like unrelated cleanup; please push that separately on trunk only. + /* We can also have: + + template typename X> + void

[PATCH] c++: ICE with TTP [PR96097]

2024-09-03 Thread Marek Polacek
Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for trunk/14? -- >8 -- We crash when dependent_type_p gets a TEMPLATE_TYPE_PARM outside a template. That happens here because in template typename X> void func() {} template struct Y {}; void g() { func(); } when performing overload