[Vm-dev] [commit][3697] Turn ARC On for 64bit compiles, don't make MIDI, fix various issues with bridge and windowIndex for ARC, fix parseArgument data type warning

[Eliot Miranda]

Hi John,

On Thu, May 5, 2016 at 10:57 AM, <commits at squeakvm.org> wrote:

>
> Revision: 3697
> Author:   johnmci
> Date:     2016-05-05 10:57:28 -0700 (Thu, 05 May 2016)
> Log Message:
> -----------
> Turn ARC On for 64bit compiles, don't make MIDI, fix various issues with
> bridge and windowIndex for ARC, fix parseArgument data type warning
>

Curious (ok, ignorant).  Why use ARC for 64-bits but weak for 32-bits?
WOup;don't it be better to go with one approach for both?  If ARC is the
future why not change the 32-bit compile to ARC too?

 [snip]

> Modified: branches/Cog/build.macos32x86/common/Makefile.flags
> ===================================================================
> --- branches/Cog/build.macos32x86/common/Makefile.flags 2016-05-05
> 17:18:10 UTC (rev 3696)
> +++ branches/Cog/build.macos32x86/common/Makefile.flags 2016-05-05
> 17:57:28 UTC (rev 3697)
> @@ -9,7 +9,7 @@
>  SDKs:=MacOSX10.12.sdk MacOSX10.11.sdk MacOSX10.10.sdk MacOSX10.9.sdk
>  SDK:=$(firstword $(realpath $(addprefix $(SDKsDIR)/, $(SDKs))))
>  TARGET_ARCH:=i386
> -TARGET_VERSION_MIN:=10.6
> +TARGET_VERSION_MIN:=10.7
>
>  CFLAGS:=$(CFLAGS) -DBUILD_FOR_OSX=1 \
>                 -arch $(TARGET_ARCH) \
> @@ -17,6 +17,7 @@
>                         -fvisibility=default -fwrapv \
>                         -fmacro-backtrace-limit=0
> -fdiagnostics-show-note-include-stack \
>                         -fmessage-length=0 -fpascal-strings -fasm-blocks
> -fstrict-aliasing \
> +                       -fobjc-weak \
>                 -isysroot $(SDK) \
>                 -include $(PLATDIR)/iOS/vm/SqueakPureObjc_Prefix.pch

[snip]

> Modified: branches/Cog/build.macos64x64/common/Makefile.flags
> ===================================================================
> --- branches/Cog/build.macos64x64/common/Makefile.flags 2016-05-05
> 17:18:10 UTC (rev 3696)
> +++ branches/Cog/build.macos64x64/common/Makefile.flags 2016-05-05
> 17:57:28 UTC (rev 3697)
> @@ -9,7 +9,7 @@
>  SDKs:=MacOSX10.12.sdk MacOSX10.11.sdk MacOSX10.10.sdk MacOSX10.9.sdk
>  SDK:=$(firstword $(realpath $(addprefix $(SDKsDIR)/, $(SDKs))))
>  TARGET_ARCH:=x86_64
> -TARGET_VERSION_MIN:=10.6
> +TARGET_VERSION_MIN:=10.7
>
>  CFLAGS:=$(CFLAGS) -DBUILD_FOR_OSX=1 \
>                 -arch $(TARGET_ARCH) \
> @@ -17,6 +17,7 @@
>                         -fvisibility=default -fwrapv \
>                         -fmacro-backtrace-limit=0
> -fdiagnostics-show-note-include-stack \
>                         -fmessage-length=0 -fpascal-strings -fasm-blocks
> -fstrict-aliasing \
> +                       -fobjc-arc \
>                 -isysroot $(SDK) \
>                 -include $(PLATDIR)/iOS/vm/SqueakPureObjc_Prefix.pch
>
[snip]

_,,,^..^,,,_
best, Eliot
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/vm-dev/attachments/20160505/52e5ae59/attachment-0001.htm