<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Thanks. I also found this very good page:<div><br></div><div><a href="http://www.m2osw.com/fr/swf_alexref.html">http://www.m2osw.com/fr/swf_alexref.html</a><div><br><div><div>On Jul 24, 2008, at 7:00 PM, P T Withington wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Approved. Rats on the no new instructions!<br><br>Possibly worth investigating for more info on swf8 byte codes: <a href="http://www.mtasc.org/">http://www.mtasc.org/</a><br><br>On 2008-07-24, at 17:50EDT, Donald Anderson wrote:<br><br><blockquote type="cite">Change 20080724-dda-2 by <a href="mailto:dda@lester.local">dda@lester.local</a> on 2008-07-24 16:43:04 EDT<br></blockquote><blockquote type="cite"> in /Users/dda/laszlo/src/svn/openlaszlo/trunk-a<br></blockquote><blockquote type="cite"> for <a href="http://svn.openlaszlo.org/openlaszlo/trunk">http://svn.openlaszlo.org/openlaszlo/trunk</a><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Summary: minor cleanup of SWF6 compiler transforms<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">New Features:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Bugs Fixed: LPP-6669 (Remove swf6 byte-code kludges from CodeGenerator.java)<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Technical Reviewer: ptw (pending)<br></blockquote><blockquote type="cite">QA Reviewer: (pending)<br></blockquote><blockquote type="cite">Doc Reviewer: (pending)<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Documentation:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Release Notes:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Details:<br></blockquote><blockquote type="cite">Most of the issues implied by the LPP-6669 cannot be fixed - but details are listed here.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">0) Removed some code specific to swf6 involving using lower case for variable and register names<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Examined and tested the various transforms in question:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">1)<br></blockquote><blockquote type="cite"> // swf6 returns undefined for comparisons with NaN, it<br></blockquote><blockquote type="cite"> // is supposed to return false (note that you cannot<br></blockquote><blockquote type="cite"> // eliminate one NOT by inverting the sense of the<br></blockquote><blockquote type="cite"> // comparison<br></blockquote><blockquote type="cite"> BinopInstrs.put(ParserConstants.LT, new Instruction[] {Instructions.LessThan, Instructions.NOT, Instructions.NOT});<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Tried setting this to just Instructions.LesThan, but smokecheck gives errors on NaN comparisons.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">2)<br></blockquote><blockquote type="cite"> // swf6 does not have GE or LE, but inverting the<br></blockquote><blockquote type="cite"> // complement operator does not work for NaN ordering<br></blockquote><blockquote type="cite"> // Luckily, LogicalOr coerces undefined to false, so we<br></blockquote><blockquote type="cite"> // don't have to play the NOT NOT trick above<br></blockquote><blockquote type="cite"> BinopInstrs.put(ParserConstants.LE, new Instruction[]<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Tried setting LE to { Instructions.GT, Instructions.NOT }<br></blockquote><blockquote type="cite"> smokecheck also gives errors on NaN comparisons.<br></blockquote><blockquote type="cite">For kicks, also tried setting LE to { Instructions.GT, Instructions.NOT , Instructions.NOT , Instructions.NOT }<br></blockquote><blockquote type="cite"> to see if the 'double NOT' trick might work, but also gives the same smokecheck errors.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Also examined flasm source to see if GE or LE has been added (they have not).<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">3)<br></blockquote><blockquote type="cite"> // swf6 does not have NE or SNE either, but inverting<br></blockquote><blockquote type="cite"> // the complement is correct for NaN<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">There are no new opcodes for NE or SNE (at least as known to flasm)<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">4)<br></blockquote><blockquote type="cite"> // Approximate a in b as b.a =! void 0<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">No new opcodes for 'in'<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">5) Special case of 'is' in translateBinaryExpression.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">No new opcodes for 'is'<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">6) Special case of 'cast' in translateBinaryExpression.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">There is a SWF 'CAST' operator that leaves a 'null' on the stack in case of error.<br></blockquote><blockquote type="cite">This is not being utilized, and there's already a separate TODO for this item.<br></blockquote><blockquote type="cite">It didn't seem to be under the purvue of this Jira bug.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Tests:<br></blockquote><blockquote type="cite"> regression: smokecheck swf8/dhtml<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Files:<br></blockquote><blockquote type="cite">M WEB-INF/lps/server/src/org/openlaszlo/sc/CodeGenerator.java<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Changeset: <a href="http://svn.openlaszlo.org/openlaszlo/patches/20080724-dda-2.tar">http://svn.openlaszlo.org/openlaszlo/patches/20080724-dda-2.tar</a><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">--<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Don Anderson<br></blockquote><blockquote type="cite">Java/C/C++, Berkeley DB, systems consultant<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">voice: 617-547-7881<br></blockquote><blockquote type="cite">email: <a href="mailto:dda@ddanderson.com">dda@ddanderson.com</a><br></blockquote><blockquote type="cite">www: <a href="http://www.ddanderson.com">http://www.ddanderson.com</a><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"><br></blockquote><br></div></blockquote></div><br><div> <span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><br class="Apple-interchange-newline">--</div><div><br>Don Anderson<br>Java/C/C++, Berkeley DB, systems consultant<br><br>voice: 617-547-7881<br>email: <a href="mailto:dda@ddanderson.com">dda@ddanderson.com</a><br>www: <a href="http://www.ddanderson.com/">http://www.ddanderson.com</a><br></div></div></span></span><br class="Apple-interchange-newline"></div></span></div></span><br class="Apple-interchange-newline"> </div><br></div></div></body></html>