[muscle] Re: MuscleSupport.h on intel Mac

  • From: Jeremy Friesner <jaf@xxxxxxxxxxxx>
  • To: muscle@xxxxxxxxxxxxx
  • Date: Tue, 2 Jan 2007 14:44:01 -0800

Hi Justin,

Like you, I don't have any 64-bit systems handy to test on, but from looking 
at the HISTORY.TXT file in the latest MUSCLE release (v3.24), I see the 
following:

[...]
v3.22 Released 7/5/2006
  * Added Lior Okman and Rony Gutherz' patches to re-enable compilation
    on 64-bit systems, Solaris, MSVC, and Windows 2000.

v3.11 Released 3/1/2006
  * Merged in Lior Okman's AMD-64 compatibility patch.

So hopefully everything is 64-bit compatible... perhaps Lior or Rony could 
describe their experiences with muscle in a 64-bit environment.

In any case, I recommend upgrading to the latest version if you can... just be 
sure to read the HISTORY.txt to see what the changes have been since your 
existing version, since some of them may require corresponding changes to 
your program's source code.

What version of MUSCLE are you currently using, btw?

-Jeremy

On Tuesday 02 January 2007 14:24, Justin Hammond wrote:
> 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: