[muscle] Re: MuscleSupport.h on intel Mac

  • From: Justin Hammond <justin@xxxxxxxx>
  • To: muscle@xxxxxxxxxxxxx
  • Date: Wed, 3 Jan 2007 06:24:48 +0800

Hi,
Ok.
What about 64bit intel boxes? I got reports that it wont compile there:

http://www.neostats.net/boards/viewtopic.php?t=2352

Again, I'm using a older version here, so let me know if its fixed or not?

Thanks

Justin

On 03 Jan 2007, at 5:58 AM, Jeremy Friesner wrote:

Hi Justin,

AFAIK Muscle has compiled correctly on Intel-based Macs since the v3.10 release, but I merged in your patch anyway since it makes the code more
logical (now that __APPLE__ no longer implies PowerPC)

Thanks,
Jeremy

On Tuesday 02 January 2007 13:44, Justin Hammond wrote:
Hi,
Not sure if this is in the latest code or not, but Muscle wont
compile in intel based mac's due to a macro. Below patch fixes this:

MuscleSupport.h 2006/01/26 15:30:22     3021
MuscleSupport.h 2007/01/01 14:30:31     3063
@@ -50,7 +50,8 @@

  /* If we are in an environment where known assembly is available,
make a note of that fact */
  #if defined(__GNUC__)
-# if (defined(__PPC__) || defined(__APPLE__))
+# if (defined(__PPC__))
  #  define MUSCLE_USE_POWERPC_INLINE_ASSEMBLY 1
  # elif defined(__i386__)
  #  define MUSCLE_USE_X86_INLINE_ASSEMBLY 1

I also believe that this will have problems on 64bit X86 platforms as
well, but I don't have a box to test on, so no patch :(

Justin



Other related posts: