[patch] Extend store merging to STRING_CST

[Richard Biener]

On Thu, Jul 2, 2020 at 12:27 PM Eric Botcazou <botcazou@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