This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix i?86 bootstrap (PR rtl-optimization/82044)
- From: Martin Liška <mliska at suse dot cz>
- To: Richard Biener <richard dot guenther at gmail dot com>, Richard Biener <rguenther at suse dot de>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Jeff Law <law at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 19 Dec 2017 12:26:23 +0100
- Subject: Re: [PATCH] Fix i?86 bootstrap (PR rtl-optimization/82044)
- Authentication-results: sourceware.org; auth=none
- References: <d430e94b-5ea6-0c5c-0dfc-a09e62e0fcbd@suse.cz> <20170920081519.GU1701@tucnak> <d7a78f76-ab66-09ed-06da-257038bc58bb@suse.cz> <7cff6742-bd7a-5ea2-80fb-aca74610f591@suse.cz> <7e976ae2-4aab-9abd-1990-94b9804db8f9@redhat.com> <87d09716-916c-974f-9a4e-7fa95f6b97c3@suse.cz> <20171121232720.GD14653@tucnak> <20171122003015.GE14653@tucnak> <alpine.LSU.2.20.1711220954340.12252@zhemvz.fhfr.qr> <CAFiYyc0D64PY9p5aCds6hxTRXe0FYB6cVtivvsOCp3=MJfaWTg@mail.gmail.com>
On 11/22/2017 10:05 AM, Richard Biener wrote:
> On Wed, Nov 22, 2017 at 9:54 AM, Richard Biener <rguenther@suse.de> wrote:
>> On Wed, 22 Nov 2017, Jakub Jelinek wrote:
>>
>>> On Wed, Nov 22, 2017 at 12:27:20AM +0100, Jakub Jelinek wrote:
>>>> The following patch fixes those two issues and adds similar overflow
>>>> check to record_store too (in that spot width is always non-negative, so
>>>> we don't need a special width == -1 handling).
>>>>
>>>> Bootstrapped successfully on i686-linux, ok for trunk if it passes regtest
>>>> there (and pending x86_64-linux bootstrap + regtest)?
>>>
>>> Now successfully bootstrapped/regtested on both.
>>
>> Ok.
>
> I've reverted the patch on the branch. This isn't stuff we should
> backport IMHO.
>
> Richard.
>
>> Richard.
Thank you all for fixing that. I'm sorry for that.
Martin