This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH (1/7)] New optab framework for widening multiplies
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: Andrew Stubbs <ams at codesourcery dot com>
- Cc: Bernd Schmidt <bernds at codesourcery dot com>, gcc-patches at gcc dot gnu dot org, patches at linaro dot org
- Date: Fri, 19 Aug 2011 16:45:02 +0200
- Subject: Re: [PATCH (1/7)] New optab framework for widening multiplies
- References: <4E034EF2.3070503@codesourcery.com> <4E034F98.5000809@codesourcery.com> <4E186929.9080805@codesourcery.com> <4E296E70.1040207@codesourcery.com> <4E297ABE.6040401@codesourcery.com> <4E297B04.3000701@codesourcery.com> <4E29988A.103@codesourcery.com> <4E4E70C3.4010604@codesourcery.com>
On Fri, Aug 19, 2011 at 4:18 PM, Andrew Stubbs <ams@codesourcery.com> wrote:
> On 22/07/11 16:34, Andrew Stubbs wrote:
>>
>> On 22/07/11 14:28, Bernd Schmidt wrote:
>>>
>>> Oh well, let's shelve it and do it later.
>>
>> Here's an updated patch with the formatting problem you found fixed.
>
> I've just committed an updated version of this patch (attached).
>
> I found a number of subtle bugs while I was testing, and these have now been
> corrected. In particular, I found that VOIDmode constants were not handled
> correctly; I've added a function "widened_mode" along the lines originally
> suggested by Benrd to deal with this. I also found one case where different
> code was produced to previously, although it was actually corrected later in
> the patch series I've fixed it here now.
Seems one in the series has broken bootstrap on x86_64 when building
the 32bit libgcc multilib in stage1.
Richard.
> Andrew
>
>