gcc 3.4.2 cpu32 patch

[Ralf Corsepius]

On Wed, 2004-11-17 at 16:25 -0800, Aaron J. Grier wrote:
> On Thu, Nov 18, 2004 at 10:58:21AM +1100, Chris Johns wrote:
> > Aaron J. Grier wrote:
> > >attached is a patch to gcc 3.4.2 to fix added preprocessor #defines
> > >for cpu32 targets.  I've included comments for clarity.
> > >
> > >I've successfully built an rtems-m68k cross-compiler for my 68331
> > >target with this patch.  (I was using peter barada's gcc 3.4.2 CVS
> > >repository, but this patch should apply to the trunk version as
> > >well.)
> > >
> > 
> > Is the patch fixing a code gen problem in GCC for the cpu32 ? Just
> > wishing clarify the purpose of the patch.
> 
> not code generation; just macro definition, as you've discovered.
> 
> > If so it maybe a good idea to raise a GCC bug and to attach the patch.
> 
> I was hoping it could be put into an "omnibus" patch from either ralf
> (on the RTEMS end) or peter (on the m68k/coldfire end).
FWIW: I am not sufficiently familiar with the m68k to be able to judge
if your patch actually is technically correct.

However, based on the comments in your patch and from what I see in the
GCC's sources, this patch seems to make sense to me.

I'd suggest you to propose this patch on gcc-patches at gcc.gnu.org and/or
file a PR in GCC's bugzilla.

Ralf