This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR80222
- From: Eric Botcazou <ebotcazou at adacore dot com>
- To: Richard Biener <rguenther at suse dot de>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sat, 13 May 2017 11:39:58 +0200
- Subject: Re: [PATCH] Fix PR80222
- Authentication-results: sourceware.org; auth=none
- References: <alpine.LSU.2.20.1703281207200.30051@zhemvz.fhfr.qr> <4531407.2ddBfSlUQt@polaris> <E627A6F0-BA82-4B27-9C23-C0F2E4C16E02@suse.de>
> Does this happen on the GCC7 branch as well? The patch just guards an
> indirect ref folding (I refrained from trying to make it correct given I
> think it's premature optimization).
No, mainline and GCC 7 branch are fine. It appears that the folding (probably
to BIT_FIELD_REF) is necessary to break apart the store on the 6 branch.
> I'll try to investigate on Monday if you don't beat me to it. Feel free to
> revert the backport in the meantime.
No urgency, it's rather marginal.
> Note I think you can trigger the same bug with some source changes
> independent of the patch which means the GENERIC must be somehow invalid.
Possibly so, yes.
--
Eric Botcazou