Details

    • Type: Improvement
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects versions: None
    • Fix versions: None
    • Labels:

      Description

      We have *warn-on-reflection* enabled for the cljs.analyzer and cljs.compiler namespaces.

      This ticket asks that this also be enabled on other large, performance-critical namespaces, like cljs.closure, where reasonable.

      This would help silly perf issues. For example this one call, when revised to avoid reflection by using clojure.string/starts-with?, improves the speed of indexing node_modules by a factor of 2: https://github.com/clojure/clojurescript/blob/132d3aa232921a3cea66f830d61c89be78c581cb/src/main/clojure/cljs/closure.clj#L2464

        Attachments

          Activity

            People

            • Assignee:
              mfikes Mike Fikes
              Reporter:
              mfikes Mike Fikes
            • Votes:
              4 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: