tree 30042058ccb2e587ae492343c91aad7652aac860
parent e05eee7d548686a12903f2905d572fd0f7b6a14b
author Marcel van Lohuizen <mpvl@golang.org> 1601976758 +0200
committer Marcel van Lohuizen <mpvl@golang.org> 1603094557 +0000

internal/core/eval: reduce per-node allocations

Removed resultNode from nodeShared and reuse
allocations of nodeShared and nodeContext.

Reduces running time of all tests by about 15%.

This also simplifies disjunction handling and prepares
for optimizing disjunction handling further.

Note that with this change, it would be possible to
merge nodeShared and nodeContext into a single
type. This is left for later work.

This also delays the creation of the arcMap to
when it is really needed, avoiding another allocation
for the majority of nodes.

Change-Id: I40f39dbc5d07992cbc3490c4512caf9845ac1881
Reviewed-on: https://cue-review.googlesource.com/c/cue/+/7403
Reviewed-by: CUE cueckoo <cueckoo@gmail.com>
Reviewed-by: Marcel van Lohuizen <mpvl@golang.org>