From d1dbea91acbeb83ed1d794a74e30c57bf9163ebf Mon Sep 17 00:00:00 2001
From: Mike Gerwitz <gerwitzm@lovullo.com>
Date: Tue, 18 Jul 2017 10:14:22 -0400
Subject: [PATCH] Remove debug collection from compiled function calls

This will hopefully provide a performance boost, and is a cheap
optimization to make.  The information it collected was pretty useless
in practice.

* src/current/compiler/js-calc.xsl (compile) [c:*]:
  Do not encase calculations with function ancestors in debug
  collection functions.
---
 src/current/compiler/js-calc.xsl | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/src/current/compiler/js-calc.xsl b/src/current/compiler/js-calc.xsl
index e3f91ae7..d2d0073d 100644
--- a/src/current/compiler/js-calc.xsl
+++ b/src/current/compiler/js-calc.xsl
@@ -60,7 +60,13 @@
 </template>
 
 
-<template match="c:*" mode="compile" priority="1">
+<!--
+  Compile calculations with debug information.  This collects the
+  result of the calculation, allowing for extremely fine-grained
+  recursive study of a calculation.
+-->
+<template mode="compile" priority="1"
+          match="c:*">
   <if test="$calcc-debug = 'yes'">
     <text>( function() { var result = </text>
   </if>
@@ -81,6 +87,18 @@
 </template>
 
 
+<!--
+  Functions are invoked many times and providing information about
+  each of their invocations is of limited use, so they should not
+  collect debug information.  This saves a lot of time and memory,
+  and hopefully allows JIT engines to better optimize.
+-->
+<template mode="compile" priority="7"
+          match="c:*[ ancestor::lv:function ]">
+  <apply-templates select="." mode="compile-pre" />
+</template>
+
+
 <!--
   Begins compilation of a calculation