From: Guido G. <ag...@de...> - 2004-10-28 07:22:38
|
Hi, Bruce, since Debian is about to release and the package is not funtcional on arm at all at the moment, I'll change all #pragmas to __attribute__(packed) for the debian package version 5.32-2. Gcc people suggest using #pragma whenever possible so we should probably do that whenever we compile with gcc. -- Guido On Thu, Oct 28, 2004 at 01:46:24AM -0500, Bruce Allen wrote: > On Thu, 28 Oct 2004 ar...@ly... wrote: > > > Bruce Allen <ba...@gr...> wrote: > > > > > > I think that this is a gcc bug. > > > Could you please compile the stock code adding: > > > -Wunknown-pragmas > > > to CFLAGS and see if it complains under ARM? > > > > > > > -Wunknown-pragmas don't cause a warning. There's more > > information in the Debian bug report here: > > > > http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=278459 > > Thank you! > > I reviewed this report and agree with the tentative conclusion that this > is a gcc bug. > > If there is something that I can put into smartmontools as a workaround > until gcc gets fixed, I'd be very happy to include it. Unfortunately the > __attribute__(packed) construction is not portable. Could you either > propose a portable contruction that fixes the ARM problem or alternatively > submit a patch which protects this with some type of > > #ifdef __GCC__ && __ARM__ /* warning: GUESSING syntax */ > > or something similar? > > Cheers, > Bruce > |