Re: [C++ Patch] PR 68087

2015-11-26 Thread Richard Biener
On Wed, Nov 25, 2015 at 5:09 PM, Paolo Carlini wrote: > Hi, > > On 11/25/2015 04:59 PM, Markus Trippelsdorf wrote: >>> >>> Index: cp/constexpr.c >>> === >>> --- cp/constexpr.c (revision 230865) >>> +++ cp/constexpr.c (workin

Re: [C++ Patch] PR 68087

2015-11-25 Thread Jason Merrill
Both are OK. Jason

Re: [C++ Patch] PR 68087

2015-11-25 Thread Paolo Carlini
Hi, On 11/25/2015 04:59 PM, Markus Trippelsdorf wrote: Index: cp/constexpr.c === --- cp/constexpr.c (revision 230865) +++ cp/constexpr.c (working copy) @@ -1799,8 +1799,8 @@ cxx_eval_array_reference (const constexpr_ctx *c

Re: [C++ Patch] PR 68087

2015-11-25 Thread Markus Trippelsdorf
On 2015.11.25 at 16:43 +0100, Paolo Carlini wrote: > Hi, > > for this small 5/6 regression (an ICE) Markus suggests in the audit trail to > protect the tree_to_shwi call with tree_fits_shwi_p. I tweaked a bit his > suggestion adopting a pattern already used, eg, in the parser, and > successfully t