This is the mail archive of the
libstdc++@gcc.gnu.org
mailing list for the libstdc++ project.
Re: [Patch] [add changelog] reduce template instantiation depth in <variant>
- From: Tim Shen <timshen at google dot com>
- To: Daniel Krügler <daniel dot kruegler at gmail dot com>
- Cc: Barrett Adair <barrettellisadair at gmail dot com>, "libstdc++" <libstdc++ at gcc dot gnu dot org>
- Date: Sat, 12 Nov 2016 12:39:28 -0800
- Subject: Re: [Patch] [add changelog] reduce template instantiation depth in <variant>
- Authentication-results: sourceware.org; auth=none
- References: <CAARDWz9UGmpwNiRu5NC_SriyzNVqJkWo27B9P7o__RgCo4Prmg@mail.gmail.com> <CAGNvRgByw+RAKGtcPRvUpcPpoRoA4dk=FCxS4F_uCTbD3YVGWg@mail.gmail.com>
On Sat, Nov 12, 2016 at 4:28 AM, Daniel Krügler
<daniel.kruegler@gmail.com> wrote:
> 2016-11-12 10:04 GMT+01:00 Barrett Adair <barrettellisadair@gmail.com>:
>>>Currently, std::variant exceeds the default ftemplate-depth parameter when instantiated with 297 types. This small patch increases this ceiling to 446 types (from the bottom of the template stack).
>>
>> Sorry, first patch - I just read the guidelines. I changed the format
>> and added a changelog entry. I hope I did it correctly this time.
>
> Let me add that this patch suggestion causes a change in semantics in
> theory, because fold expressions have no short-circuit evaluation,
Is it a compiler QoI problem, or part of the language? I'd be
surprised if the language somehow prohibits the short circuits from
happening.
> contrary to __and_. Whether this difference is relevant here is
> of-course something the maintainer has to decide.
>
> - Daniel
--
Regards,
Tim Shen