This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA:] Fix PR optimization/15296, delayed-branch-slot bug.


> Date: Wed, 5 May 2004 18:54:17 +0200
> From: Hans-Peter Nilsson <hp@axis.com>
> > From: law@redhat.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
policies".)

brgds, H-P


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]