This is the mail archive of the gcc@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: libgcc fails to compile if DItype is not supported [bswapdi2]

From: Eric Christopher <echristo at apple dot com>
To: Pompapathi V Gadad <Pompapathi dot V dot Gadad at nsc dot com>
Cc: gcc at gcc dot gnu dot org
Date: Wed, 20 Jun 2007 16:52:32 -0700
Subject: Re: libgcc fails to compile if DItype is not supported [bswapdi2]
References: <4678C015.50106@nsc.com>

On Jun 19, 2007, at 10:50 PM, Pompapathi V Gadad wrote:

Hello,
Current function declaration of __bswapdi2 in libgcc2.h is:
DItype __bswapdi2 (DItype u)
Since this declaration does not check if DItype is supported, it is bound for compilation failure for targets that do not support DItype. Would it be ok to change the DItype to DWtype as in:

DWtype __bswapdi2 (DWtype u)

Is the above declaration more safer for all targets?

No, probably best would be a check to make sure DItype is defined. In this case it's not a double word operation it is explicitly a 64-bit operation.

-eric

Follow-Ups:
- Re: libgcc fails to compile if DItype is not supported [bswapdi2]
  - From: Pompapathi V Gadad

References:
- libgcc fails to compile if DItype is not supported [bswapdi2]
  - From: Pompapathi V Gadad

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