Never mind, I guess that line evaluates to true because you're compiling for x86 at the same time. I overlooked the "Universal Binary" in the subject heading. If you have any suggestions on how to fix the makefile and/or config.h to better support universal binary, I'd welcome them.
BTW, -DCRYPTOPP_DISABLE_ASM is auto-added because Apple's version of GAS doesn't support .intel_syntax. Will they ever upgrade to a newer version of GAS that does support it, I wonder? -------------------------------------------------- From: "Wei Dai" <[email protected]> Sent: Friday, May 22, 2009 2:57 PM To: "Gianluca Toso" <[email protected]>; "Crypto++ Users" <[email protected]> Subject: Re: Crypto++ Universal Binary for Mac > > Can you post the output of "echo | g++ -E -dM -" on PPC? Assembly should > only be turned on only if: > > #if !defined(CRYPTOPP_DISABLE_ASM) && ((defined(_MSC_VER) && > defined(_M_IX86)) || (defined(__GNUC__) && (defined(__i386__) || > defined(__x86_64__)))) > > Why are these defined on PPC? > > -------------------------------------------------- > From: "Gianluca Toso" <[email protected]> > Sent: Friday, May 22, 2009 7:32 AM > To: "Crypto++ Users" <[email protected]> > Subject: Re: Crypto++ Universal Binary for Mac > >> >> On 14 Mag, 14:56, Sancio <[email protected]> wrote: >>> On May 6, 7:28 am, Mirko Grewing <[email protected]> wrote: >> >>> > I'm under Intel Mac so I didn't use -DCRYPTOPP_DISABLE_ASM option. >>> >>> I confirm, >>> if you compile under PCC instead you need it do add it. >> >> A clarification: >> On Intel -DCRYPTOPP_DISABLE_ASM is not needed because auto added due >> to as version (1.38 on Leopard). >> On PPC you need to add it because as tests are not executed even when >> universal compile (uname -m is Power Macintosh). >> >> IMHO in the next version some fixes to the makefile are needed. >> >> > >> > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the "Crypto++ Users" Google Group. To unsubscribe, send an email to [email protected]. More information about Crypto++ and this group is available at http://www.cryptopp.com. -~----------~----~----~----~------~----~------~--~---
