Re: [patch, 4.3] Fix PR middle-end/38616

[Richard Guenther <richard dot guenther at gmail dot com>]

On Mon, Jan 19, 2009 at 12:06 AM, Matthias Klose <doko@cs.tu-berlin.de> wrote:
> H.J. Lu writes:
>> On Sun, Jan 11, 2009 at 7:18 AM, Matthias Klose <doko@cs.tu-berlin.de> wrote:
>> > The backport was suggested in the bug report; checked on
>> > i486-linux-gnu with a biarch compiler without regressions.
>> >
>> > Ok for the branch (and the testcase for the trunk)?
>> >
>> >  Matthias
>> >
>> > gcc/
>> >
>> > 2009-01-11  Matthias Klose  <doko@ubuntu.com>
>> >
>> >        PR middle-end/38616, backport from mainline:
>> >        2008-05-04  Uros Bizjak  <ubizjak@gmail.com>
>> >
>> >        * config/i386/i386.md (*strmovsi_1): Simplify asm alternatives.
>> >        (*strmovsi_rex_1): Ditto.
>> >        (*strsetsi_1): Ditto.
>> >        (*strsetsi_rex_1): Ditto.
>> >
>> >        (add<mode>cc): Macroize expander from addqicc, addhicc, addsicc and
>> >        adddicc expanders using SWI mode iterator.
>> >
>> > gcc/testsuite/
>> >
>> > 2009-01-11  Matthias Klose  <doko@ubuntu.com>
>> >
>> >        PR middle-end/38616
>> >        * gcc.dg/pr38616.c: New test.
>> >
>>
>> I don't think this back port is sufficient to fix this. On
>> Fedora 10/x86, I saw
>>
>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38902
>
> confirmed.  I rechecked, and I have to say that the patch doesn't fix
> the problem.  I cannot reproduce what I made wrong with testing the
> patch. Sorry about that.

Can you revert the testcase (just the testcase) on the branch for now?

Thanks,
Richard.