@markw65/monkeyc-optimizer 1.1.23 → 1.1.25

package/README.md

@@ -828,3 +828,23 @@ Bug Fixes
 ### 1.1.23
 
 No functional change, just fixes a typo that broke the typescript exports.
+
+### 1.1.24
+
+- Bug fixes
+
+  - Conversion of unary `-x` to `0 - x` was too restrictive, causing some missed optimization opportunities
+
+- Post Build Optimizations
+  - Added a pass equivalent to `Minimize Locals` in the source-to-source optimizer. This pass can see and re-assign all the locals, so it does better than the source-to-source optimizer. In addition, it maintains the variable mapping, so the debugger still sees the original names. As a result, its probably best to disable the `Minimize Locals` pass if the post build optimizer is enabled.
+  - Added a pass similar to `Size Based PRE` in the source-to-source optimizer. Currently this only optimizes constants and Symbols, but it has visibility to a lot of things the source-to-source optimizer can't see; so the two passes are complementary. I've added an option in case it causes problems, but it's enabled by default when the post build optimizer is enabled.
+  - Added various new optimizations to the interp pass:
+    - Handles a few more byte codes
+    - Conditional branches that are known to be taken, or known to be not taken are converted to gotos (and the gotos will often be eliminated by re-ordering blocks)
+    - Conditional branches that would be known to be taken, or known to be not taken if evalated at the end of one of their predecessors will be bypassed from that predecessor. Amongst other things, this converts for and while loops, that can be proven to iterate at least once, into do-while loops.
+  - Improved the emitter's algorithm for ordering blocks to avoid some more gotos
+
+### 1.1.25
+
+- Bug fixes
+  - fixes a copy paste error that could cause strange results in the interpreter, possibly leading to incorrect optimizations.

package/build/api.cjs

@@ -18,42 +18,42 @@ var __copyProps = (to, from, except, desc) => {
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 var api_exports = {};
 __export(api_exports, {
-  checkCompilerVersion: () => import_chunk_27ZJJM2Y.checkCompilerVersion,
-  collectNamespaces: () => import_chunk_27ZJJM2Y.collectNamespaces,
-  createDocumentationMap: () => import_chunk_27ZJJM2Y.createDocumentationMap,
-  diagnostic: () => import_chunk_27ZJJM2Y.diagnostic,
-  diagnosticHelper: () => import_chunk_27ZJJM2Y.diagnosticHelper,
-  findNamesInScope: () => import_chunk_27ZJJM2Y.findNamesInScope,
-  findUsingForNode: () => import_chunk_27ZJJM2Y.findUsingForNode,
-  formatAst: () => import_chunk_27ZJJM2Y.formatAst,
-  formatAstLongLines: () => import_chunk_27ZJJM2Y.formatAstLongLines,
-  getApiFunctionInfo: () => import_chunk_27ZJJM2Y.getApiFunctionInfo,
-  getApiMapping: () => import_chunk_27ZJJM2Y.getApiMapping,
-  getSuperClasses: () => import_chunk_27ZJJM2Y.getSuperClasses,
-  hasProperty: () => import_chunk_27ZJJM2Y.hasProperty,
-  isClassVariable: () => import_chunk_27ZJJM2Y.isClassVariable,
-  isLocal: () => import_chunk_27ZJJM2Y.isLocal,
-  isLookupCandidate: () => import_chunk_27ZJJM2Y.isLookupCandidate,
-  isStateNode: () => import_chunk_27ZJJM2Y.isStateNode,
-  lookupByFullName: () => import_chunk_27ZJJM2Y.lookupByFullName,
-  lookupNext: () => import_chunk_27ZJJM2Y.lookupNext,
-  lookupResultContains: () => import_chunk_27ZJJM2Y.lookupResultContains,
-  lookupWithType: () => import_chunk_27ZJJM2Y.lookupWithType,
-  makeToyboxLink: () => import_chunk_27ZJJM2Y.makeToyboxLink,
-  mapVarDeclsByType: () => import_chunk_27ZJJM2Y.mapVarDeclsByType,
-  markInvokeClassMethod: () => import_chunk_27ZJJM2Y.markInvokeClassMethod,
-  parseSdkVersion: () => import_chunk_27ZJJM2Y.parseSdkVersion,
-  sameLookupResult: () => import_chunk_27ZJJM2Y.sameLookupResult,
-  traverseAst: () => import_chunk_27ZJJM2Y.traverseAst,
-  variableDeclarationName: () => import_chunk_27ZJJM2Y.variableDeclarationName,
-  visitReferences: () => import_chunk_27ZJJM2Y.visitReferences,
-  visit_resources: () => import_chunk_27ZJJM2Y.visit_resources,
-  visitorNode: () => import_chunk_27ZJJM2Y.visitorNode
+  checkCompilerVersion: () => import_chunk_SNWJ2FZL.checkCompilerVersion,
+  collectNamespaces: () => import_chunk_SNWJ2FZL.collectNamespaces,
+  createDocumentationMap: () => import_chunk_SNWJ2FZL.createDocumentationMap,
+  diagnostic: () => import_chunk_SNWJ2FZL.diagnostic,
+  diagnosticHelper: () => import_chunk_SNWJ2FZL.diagnosticHelper,
+  findNamesInScope: () => import_chunk_SNWJ2FZL.findNamesInScope,
+  findUsingForNode: () => import_chunk_SNWJ2FZL.findUsingForNode,
+  formatAst: () => import_chunk_SNWJ2FZL.formatAst,
+  formatAstLongLines: () => import_chunk_SNWJ2FZL.formatAstLongLines,
+  getApiFunctionInfo: () => import_chunk_SNWJ2FZL.getApiFunctionInfo,
+  getApiMapping: () => import_chunk_SNWJ2FZL.getApiMapping,
+  getSuperClasses: () => import_chunk_SNWJ2FZL.getSuperClasses,
+  hasProperty: () => import_chunk_SNWJ2FZL.hasProperty,
+  isClassVariable: () => import_chunk_SNWJ2FZL.isClassVariable,
+  isLocal: () => import_chunk_SNWJ2FZL.isLocal,
+  isLookupCandidate: () => import_chunk_SNWJ2FZL.isLookupCandidate,
+  isStateNode: () => import_chunk_SNWJ2FZL.isStateNode,
+  lookupByFullName: () => import_chunk_SNWJ2FZL.lookupByFullName,
+  lookupNext: () => import_chunk_SNWJ2FZL.lookupNext,
+  lookupResultContains: () => import_chunk_SNWJ2FZL.lookupResultContains,
+  lookupWithType: () => import_chunk_SNWJ2FZL.lookupWithType,
+  makeToyboxLink: () => import_chunk_SNWJ2FZL.makeToyboxLink,
+  mapVarDeclsByType: () => import_chunk_SNWJ2FZL.mapVarDeclsByType,
+  markInvokeClassMethod: () => import_chunk_SNWJ2FZL.markInvokeClassMethod,
+  parseSdkVersion: () => import_chunk_SNWJ2FZL.parseSdkVersion,
+  sameLookupResult: () => import_chunk_SNWJ2FZL.sameLookupResult,
+  traverseAst: () => import_chunk_SNWJ2FZL.traverseAst,
+  variableDeclarationName: () => import_chunk_SNWJ2FZL.variableDeclarationName,
+  visitReferences: () => import_chunk_SNWJ2FZL.visitReferences,
+  visit_resources: () => import_chunk_SNWJ2FZL.visit_resources,
+  visitorNode: () => import_chunk_SNWJ2FZL.visitorNode
 });
 module.exports = __toCommonJS(api_exports);
-var import_chunk_27ZJJM2Y = require("./chunk-27ZJJM2Y.cjs");
-var import_chunk_HHQDDCTP = require("./chunk-HHQDDCTP.cjs");
-(0, import_chunk_27ZJJM2Y.init_api)();
+var import_chunk_SNWJ2FZL = require("./chunk-SNWJ2FZL.cjs");
+var import_chunk_QFQPBQSO = require("./chunk-QFQPBQSO.cjs");
+(0, import_chunk_SNWJ2FZL.init_api)();
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   checkCompilerVersion,

package/build/{chunk-HHQDDCTP.cjs → chunk-QFQPBQSO.cjs}

@@ -26,8 +26,8 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
   mod
 ));
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
-var chunk_HHQDDCTP_exports = {};
-__export(chunk_HHQDDCTP_exports, {
+var chunk_QFQPBQSO_exports = {};
+__export(chunk_QFQPBQSO_exports, {
   GenericQueue: () => GenericQueue,
   __commonJS: () => __commonJS,
   __esm: () => __esm,
@@ -58,7 +58,7 @@ __export(chunk_HHQDDCTP_exports, {
   spawnByLine: () => spawnByLine,
   wouldLog: () => wouldLog
 });
-module.exports = __toCommonJS(chunk_HHQDDCTP_exports);
+module.exports = __toCommonJS(chunk_QFQPBQSO_exports);
 var child_process = __toESM(require("child_process"));
 var fsc = __toESM(require("fs"));
 var fs = __toESM(require("fs/promises"));
@@ -5455,7 +5455,7 @@ var require_priorityqueuejs = __commonJS({
 });
 function logger(module2, level, message) {
   if (wouldLog(module2, level)) {
-    log(message);
+    log(typeof message === "function" ? message() : message);
   }
 }
 function wouldLog(module2, level) {