search

use-strict / 7z-wasm

7-Zip for JavaScript environments, compiled to WASM
Other
109 stars 7 forks source link

Link with -O2 flag #19

Closed kichikuou closed 11 months ago

kichikuou commented 1 year ago

Currently, cpp files are compiled with the -O2 flag, but no optimization flag is specified when linking.

This patch adds the -O2 flag to link times, which significantly reduces code size (mainly due to JS minification).

kichikuou commented 1 year ago

As for speed, in my environment this made unpacking a 7z archive about 20% faster.

use-strict commented 1 year ago

LGTM. I will need to add some tests though before merging this, just to be safe.

use-strict commented 11 months ago

@kichikuou , please update your branch to the latest upstream master

kichikuou commented 11 months ago

@kichikuou , please update your branch to the latest upstream master

Rebased and force-pushed.