This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix for PR 37809 and 37807
- From: "Richard Guenther" <richard dot guenther at gmail dot com>
- To: "Ralph Loader" <suckfish at ihug dot co dot nz>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 5 Nov 2008 10:20:50 +0100
- Subject: Re: [PATCH] Fix for PR 37809 and 37807
- References: <20081022181234.2acf7548@i.geek.nz> <84fc9c000810220316t20257102t9ce3b53e95d3400b@mail.gmail.com> <20081105205605.20d170c4@i.geek.nz>
On Wed, Nov 5, 2008 at 8:56 AM, Ralph Loader <suckfish@ihug.co.nz> wrote:
> [Sorry for the delay on following up on this]
>
>> The rtlanal.c parts are ok. I wonder if we ever hit the _or_truncate
>> case with vector modes in force_to_mode though - can you check if
>> using gen_lowpart instead of gen_lowpart_or_truncate works?
>
> I checked this and gen_lowpart instead of gen_lowpart_or_truncate
> appears fine.
>
> Attached is patch modified to use gen_lowpart, and with the test case
> cleaned up & changeloged.
The patch is ok this way.
Thanks,
Richard.
> Cheers,
> Ralph.
>
>
>>
>> Btw, you missa ChangeLog entry for the testcase for
>> testsuite/ChangeLog.
>>
>> Thanks,
>> Richard.
>>
>> > 2008-10-19 Ralph Loader <suckfish@ihug.co.nz>
>> >
>> > PR middle-end/37807, middle-end/37809
>> > * combine.c (force_to_mode): Do not process vector types.
>> >
>> > * rtlanal.c (nonzero_bits1): Do not process vector types.
>> > (num_sign_bit_copies1): Likewise.
>> >
>> > Cheers,
>> > Ralph.
>> >
>