On Thu, Jul 2, 2020 at 12:27 PM Eric Botcazou <botca...@adacore.com> wrote: > > > Sorry for the ping-pong but why's using a new char[len] type problematic? > > Because the type may be incompatible with that of the STRING_CST, so we would > need to build a new STRING_CST with the new type.
I see. > > That said, I do like p2 more even if we need to special-case STRING_CST > > sources at the end for store-merging to be happy. > > OK. So this variant combined with the rest of the patch is OK then. Thanks, Richard. > > -- > Eric Botcazou