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

Reply via email to