Skip to content

Commit e3d5631

Browse files
authored
Revise default optimization level (#1776)
BREAKING: Changes default optimization levels for `--optimize` respectively `-O` to `optimizeLevel=3` and `shrinkLevel=0`, which is more useful in practice.
1 parent 1c55d37 commit e3d5631

File tree

103 files changed

+208606
-135549
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

103 files changed

+208606
-135549
lines changed

cli/asc.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ exports.libraryPrefix = __getString(assemblyscript.LIBRARY_PREFIX.valueOf());
171171
exports.defaultOptimizeLevel = 3;
172172

173173
/** Default Binaryen shrink level. */
174-
exports.defaultShrinkLevel = 1;
174+
exports.defaultShrinkLevel = 0;
175175

176176
/** Bundled library files. */
177177
exports.libraryFiles = exports.isBundle ? BUNDLE_LIBRARY : (() => { // set up if not a bundle

cli/asc.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
"description": [
3636
"Optimizes the module. Typical shorthands are:",
3737
"",
38-
" Default optimizations -O / -O3s",
38+
" Default optimizations -O",
3939
" Make a release build -O --noAssert",
4040
" Make a debug build --debug",
4141
" Optimize for speed -Ospeed",
@@ -377,8 +377,8 @@
377377
"description": "Specifies node.js options (CLI only). See: node --help"
378378
},
379379

380-
"-Os": { "value": { "optimize": true, "shrinkLevel": 1 } },
381-
"-Oz": { "value": { "optimize": true, "shrinkLevel": 2 } },
380+
"-Os": { "value": { "optimizeLevel": 0, "shrinkLevel": 1 } },
381+
"-Oz": { "value": { "optimizeLevel": 0, "shrinkLevel": 2 } },
382382
"-O0": { "value": { "optimizeLevel": 0, "shrinkLevel": 0 } },
383383
"-O1": { "value": { "optimizeLevel": 1, "shrinkLevel": 0 } },
384384
"-O2": { "value": { "optimizeLevel": 2, "shrinkLevel": 0 } },

tests/asconfig/complicated/assembly/index.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
assert(ASC_OPTIMIZE_LEVEL == 3, "expected optimize level == 3");
2-
assert(ASC_SHRINK_LEVEL == 1, "expected shrink level == 1");
2+
assert(ASC_SHRINK_LEVEL == 0, "expected shrink level == 0");
33
assert(ASC_FEATURE_SIMD, "expected SIMD enabled");
44
let size = memory.size();
55
trace("size", 1, size);

0 commit comments

Comments
 (0)