<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 17 Jun 2015, at 07:45, Ryan Macnak <<a href="mailto:rmacnak@gmail.com" class="">rmacnak@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><div class="gmail_extra"><div class="gmail_quote">On Sat, Jun 13, 2015 at 12:17 AM, Esteban Lorenzano<span class="Apple-converted-space"> </span><span dir="ltr" class=""><<a href="mailto:estebanlm@gmail.com" target="_blank" class="">estebanlm@gmail.com</a>></span><span class="Apple-converted-space"> </span>wrote:<br class=""><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex;"> <br class=""><div style="word-wrap: break-word;" class=""><br class=""><div class=""><blockquote type="cite" class=""><div class="">On 13 Jun 2015, at 03:38, Ryan Macnak <<a href="mailto:rmacnak@gmail.com" target="_blank" class="">rmacnak@gmail.com</a>> wrote:</div><br class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 12, 2015 at 12:43 AM, Esteban Lorenzano<span class="Apple-converted-space"> </span><span dir="ltr" class=""><<a href="mailto:estebanlm@gmail.com" target="_blank" class="">estebanlm@gmail.com</a>></span><span class="Apple-converted-space"> </span>wrote:<blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex;"><div style="word-wrap: break-word;" class=""><div class=""><div class=""><div class=""><pre style="word-wrap: break-word; white-space: pre-wrap;" class="">extern extern sqInt superclassOf(sqInt classPointer);</pre></div></div></div></div></blockquote>gcc 4.6.3 (Travis CI) also says this is an error.</div><div class="gmail_quote"><br class=""></div><div class="gmail_quote">Apple LLVM version 5.1 (clang-503.0.40) (based on LLVM 3.4svn) is okay with it.<br class=""></div><div class="gmail_quote"><br class=""><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex;"><div style="word-wrap: break-word;" class=""><div class=""><div class="">ps: btw, I’m using clang to compile in mac since a lot of time and I do not have problems, yes I needed to tweak a flag here and there, but not much more… of course, I use “Apple LLVM version 6.0 (clang-600.0.54) (based on LLVM 3.5svn)”, so is probably not the same...</div></div></div></blockquote><div class=""><br class=""></div><div class="">What flags have you changed? We should adopt them or fix things so they aren't needed.</div></div></div></div></div></blockquote><br class=""></div><div class=""><br class=""></div><div class="">I’m compiling with this: </div><div class=""><br class=""></div><div class="">-D_FILE_OFFSET_BITS=64 -DNDEBUG -D_GNU_SOURCE -DLSB_FIRST -DHAVE_SYS_TIME_H -DHAVE_UUID_GENERATE -Wno-unused-parameter -mmacosx-version-min=10.5 -DHAVE_NANOSLEEP -O3 -fomit-frame-pointer -DDEBUGVM=0 -fpascal-strings -DCOGVM -Qunused-arguments -funroll-loops -DDEFAULT_IMAGE_NAME="Pharo.image" -Wno-unused-value -fmessage-length=0 -DUSE_CORE_GRAPHICS -g0 -DBASE_HEADER_SIZE=4 -march=pentium-m -DBUILD_FOR_OSX -DENABLE_FAST_BLT -fasm-blocks -Wno-unused-variable -std=gnu99 -DCOGMTVM=0 -fwrapv -Wno-return-type -fno-inline-functions -fno-gcse -mfpmath=sse -arch i386 -mtune=prescott -falign-functions=16 -DUSE_GLOBAL_STRUCT=0</div><br class=""><div class="">I think the only “CLANG Only” is: </div><div class=""><br class=""></div><div class="">-fwrapv </div></div></blockquote><div class=""><br class=""></div><div class="">That one fixes my VMs. Looks like it goes back to gcc 3.3, so it should be safe to add to all the builds.</div><div class=""><br class=""></div><div class="">But I didn't need this before. Your merge with Eliot falls within the range of commits where things stopped working. Are you aware of code from your branch that relies on signed integer overflow?</div></div></div></div></div></blockquote><div><br class=""></div><div>yes you needed :)</div><div>this flag is to say the compiler to avoid an optimisation that skips the overflow check assuming it will be ok, and the vm uses that “trick” to calculate when it has to give you a SmallInteger or a LongPositiveInteger. </div><div>clang 3.3 didn’t do those optimisations by default (I remember I didn’t needed), and then at some point of Xcode 4.x, I do not remember clang version, it started to be a problem… that’s why I added it. </div><div><br class=""></div><div>My merge with Eliot was simply dropping some code and align with current sources, and nothing I did there or in the platform files (where the main difference is) can lead to this problem.</div><div>Right now pharovm is the same as a regular spur vm (because also, my merge is in the spur branch, not in the master)… only changes are in plugins and platform files.</div><div><br class=""></div><div>cheers,</div><div>Esteban</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><div class="gmail_extra"><div class="gmail_quote"><div class=""><br class=""></div></div>Ryan</div></div></div></blockquote></div><br class=""></body></html>