Fork me on GitHub
Roman Liutikov07:05:02

@dnolen regarding lifting small data literals into constants, do we want to align with Clojure here, where data literals are cached per function or just put everything into constants table?


It probably shouldn't do what clojure does for all "constants". a) that might interfere with DCE since Closure doesn't understand the CLJS datastructures very well sometimes and b) might affect overall startup performance since it would do quite a lot when the code is loaded


also clojure inlines constants per ns, which may lead to lots of duplication


@thheller last I looked Closure did seem to understand ClojureScript constants where everything is static


that said the feature should be behind a flag that defaults to false so that folks who are familiar with tuning peformance can experiment and report back


w/o affecting typical users


@roman01la we should just reuse what have, the constants table approach was designed to aid DCE


flag probably should be called :optimize-constant-colls or something like that


Is it intentional that the webpack guide uses -v in all it's commands? The output is really loud for first run.


e.g. Reading analysis cache for jar:file:/home/overfl0w/.m2/repository/org/clojure/clojurescript/1.10.741/clojurescript-1.10.741.jar!/cljs/spec/alpha.cljs Compiling /home/overfl0w/.cljs/.aot_cache/1.10.741/35F2CCB/reagent/dom.cljs to /home/overfl0w/.cljs/.aot_cache/1.10.741/35F2CCB/reagent/dom.js Can't really see the prompt