<div dir="ltr">Hi Ryan,<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 16, 2015 at 10:45 PM, Ryan Macnak <span dir="ltr"><<a href="mailto:rmacnak@gmail.com" target="_blank">rmacnak@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <br><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Jun 13, 2015 at 12:17 AM, Esteban Lorenzano <span dir="ltr"><<a href="mailto:estebanlm@gmail.com" target="_blank">estebanlm@gmail.com</a>></span> wrote:<br><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><div style="word-wrap:break-word"><br><div><blockquote type="cite"><div>On 13 Jun 2015, at 03:38, Ryan Macnak <<a href="mailto:rmacnak@gmail.com" target="_blank">rmacnak@gmail.com</a>> wrote:</div><br><div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 12, 2015 at 12:43 AM, Esteban Lorenzano <span dir="ltr"><<a href="mailto:estebanlm@gmail.com" target="_blank">estebanlm@gmail.com</a>></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"><div><div><div><pre style="word-wrap:break-word;white-space:pre-wrap">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></div><div class="gmail_quote">Apple LLVM version 5.1 (clang-503.0.40) (based on LLVM 3.4svn) is okay with it.<br></div><div class="gmail_quote"><br><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"><div><div>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><br></div><div>What flags have you changed? We should adopt them or fix things so they aren't needed.</div></div></div></div>
</div></blockquote><br></div><div><br></div><div>I’m compiling with this: </div><div><br></div><div>-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><div>I think the only “CLANG Only” is: </div><div><br></div><div>-fwrapv </div></div></blockquote><div><br></div><div>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></div></div></blockquote><div><br></div><div>You mean just adding -fwrapv fixes your VMs, or all of the above?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>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?<br></div><div><br></div></div>Ryan</div></div>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">best,<div>Eliot</div></div>
</div></div>