Re: RFR: 8245969: Simplify String concat constant folding

2020-05-27 Thread Claes Redestad
On 2020-05-27 22:00, Paul Sandoz wrote: That’s rather elegant. Nicely done. Paul. Thanks, Paul! /Claes

Re: RFR: 8245969: Simplify String concat constant folding

2020-05-27 Thread Claes Redestad
On 2020-05-27 21:34, fo...@univ-mlv.fr wrote: http://cr.openjdk.java.net/~redestad/8245969/open.01 OK? Ok ! Thanks, Rémi! /Claes

Re: RFR: 8245969: Simplify String concat constant folding

2020-05-27 Thread Paul Sandoz
That’s rather elegant. Nicely done. Paul. > On May 27, 2020, at 8:50 AM, Claes Redestad wrote: > > Hi Rémi, > > thanks for looking at this. > > On 2020-05-27 17:12, Remi Forax wrote: >> Hi Claes, >> so instead of having a prefix and a suffix, there is only a prefix, a suffix >> being seen as

Re: RFR: 8245969: Simplify String concat constant folding

2020-05-27 Thread forax
- Mail original - > De: "Claes Redestad" > À: "Remi Forax" > Cc: "core-libs-dev" > Envoyé: Mercredi 27 Mai 2020 17:50:19 > Objet: Re: RFR: 8245969: Simplify String concat constant folding > Hi Rémi, > > thanks for looking at t

Re: RFR: 8245969: Simplify String concat constant folding

2020-05-27 Thread Claes Redestad
Hi Rémi, thanks for looking at this. On 2020-05-27 17:12, Remi Forax wrote: Hi Claes, so instead of having a prefix and a suffix, there is only a prefix, a suffix being seen as a prefix for the next iteration and if at the end instead of just allocating an array, you allocate an array and stu

Re: RFR: 8245969: Simplify String concat constant folding

2020-05-27 Thread Remi Forax
Hi Claes, so instead of having a prefix and a suffix, there is only a prefix, a suffix being seen as a prefix for the next iteration and if at the end instead of just allocating an array, you allocate an array and stuff it with the last prefix. Are you sure having a suffix at the end is uncommon