[prev in list] [next in list] [prev in thread] [next in thread] 

List:       qemu-devel
Subject:    Re: [Qemu-devel] Build failure on OS X
From:       Andreas_Färber <andreas.faerber () web ! de>
Date:       2007-09-30 15:54:55
Message-ID: 6E216391-AFD2-4037-A2DF-6AA491F9EB23 () web ! de
[Download RAW message or body]

Am 30.09.2007 um 17:05 schrieb Andreas Färber:

> Am 30.09.2007 um 16:37 schrieb J. Mayer:
>
>>> If someone has a speedup idea for the __APPLE__ case, that could
>>> still be applied separately.
>>
>> I guess there are some already defined macros in the Apple build
>> environmnet that should be used instead. But I don't know too much  
>> about
>> this environment... The slowdown is really not an issue in that
>> particular case.
>
> Actually, I am now seeing that in math.h Apple itself uses  
> __attribute__ ((always_inline)) for __MATH_H_ALWAYS_INLINE__ if  
> __GNUC__ is defined ... so I now believe the issue is rather the  
> "recursive" definition of always_inline: QEMU defines always_inline  
> and it subsequently tries to replace always_inline inside the  
> __attribute__(()) in math.h! Could this be it?

Puzzling, the attached patch works. It appears to be the trailing  
inline that it doesn't like for some reason.

Regards,

Andreas

["vl.h-exec-all.h-inline-2007-09-30.diff" (vl.h-exec-all.h-inline-2007-09-30.diff)]

Index: vl.h
===================================================================
RCS file: /sources/qemu/qemu/vl.h,v
retrieving revision 1.273
diff -r1.273 vl.h
113c113
< #if (__GNUC__ < 3) || defined(__APPLE__)
---
> #if (__GNUC__ < 3)
114a115,116
> #elif defined(__APPLE__)
> #define always_inline __attribute__ (( always_inline ))
Index: exec-all.h
===================================================================
RCS file: /sources/qemu/qemu/exec-all.h,v
retrieving revision 1.66
diff -r1.66 exec-all.h
41c41
< #if (__GNUC__ < 3) || defined(__APPLE__)
---
> #if (__GNUC__ < 3)
42a43,44
> #elif defined(__APPLE__)
> #define always_inline __attribute__ (( always_inline ))




[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic