This is the mail archive of the
mailing list for the GCC project.
Re: [RFA:] Fix PR optimization/15296, delayed-branch-slot bug.
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: gcc-patches at gcc dot gnu dot org
- Date: Fri, 7 May 2004 05:12:07 +0200
- Subject: Re: [RFA:] Fix PR optimization/15296, delayed-branch-slot bug.
> Date: Wed, 5 May 2004 18:54:17 +0200
> From: Hans-Peter Nilsson <firstname.lastname@example.org>
> > From: email@example.com
> > Date: Wed, 05 May 2004 10:20:46 -0600
> > The testcase at least at first glance looks like it may not work for our
> > 16bit targets.... You might want to restrict it to working on 32/64 bit
> > targets.
> Something in particular that looks nonworking? I believe the
> assumption that "void *" is assignment-compatible with
> "int __attribute__ ((mode (__pointer__)))" is safe, but is that all?
> I've started a h8300-elf run to see if it fails there.
It works there, as expected. I tried to build m68hc11-elf, but
that port was broken (PR target/15334) and I don't see a
baseboard for xstormy-sim. I know of no other 16-bit simulator
target. I think it would (even further) obfuscate the test-case
to add #ifdefs to restrict it to 32 and 64-bit targets, when
it's written to accomodate any pointer size, as above.
Being port maintainer, I can check in test-cases for bugs found
in targets I maintain, so I'll go ahead and check it in as-is.
(Right, cvswrite.html doesn't say exactly that, but it does say
"We don't want to get overly anal-retentive about checkin