This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] PR86844: Fix for store merging
- From: Andreas Krebbel <krebbel at linux dot ibm dot com>
- To: Jeff Law <law at redhat dot com>, Eric Botcazou <ebotcazou at adacore dot com>, Richard Biener <richard dot guenther at gmail dot com>, Jakub Jelinek <jakub at redhat dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Andreas Krebbel <krebbel at linux dot vnet dot ibm dot com>
- Date: Mon, 10 Sep 2018 16:05:26 +0200
- Subject: Re: [PATCH] PR86844: Fix for store merging
- References: <20180807113526.15097-1-krebbel@linux.ibm.com> <CAFiYyc3N76VwCp5eXefJaxWK795tCtS0cJ+GHyTQrpoH2Tr+=A@mail.gmail.com> <3468595.rsgiZXrS6R@arcturus.home> <a27ea6eb-cee6-c84b-0241-5bc67f228725@redhat.com>
On 20.08.2018 16:30, Jeff Law wrote:
> On 08/18/2018 03:20 AM, Eric Botcazou wrote:
>>> Eric, didn't your patches explicitely handle this case of a non-constant
>>> inbetween?
>>
>> Only if there is no overlap at all, otherwise you cannot do things simply.
>>
>>> Can you have a look / review here?
>>
>> Jakub is probably more qualified to give a definitive opinion, as he wrote
>> check_no_overlap and the bug is orthogonal to my patches since it is present
>> in 8.x; in any case, all transformations are supposed to be covered by the
>> testsuite.
> FYI. Jakub is on PTO through the end of this week and will probably be
> buried when he returns.
Jakub, could you please have a look whether that's the right fix?
https://gcc.gnu.org/ml/gcc-patches/2018-08/msg00474.html
Andreas