bug366, discussion (missed optimization)

Percentage Accurate: 52.8% → 76.2%
Time: 1.9s
Alternatives: 2
Speedup: 1.0×

Specification

?
\[\begin{array}{l} \\ \sqrt{a \cdot a - b \cdot b} \end{array} \]
(FPCore (a b) :precision binary64 (sqrt (- (* a a) (* b b))))
double code(double a, double b) {
	return sqrt(((a * a) - (b * b)));
}

real(8) function code(a, b)
    real(8), intent (in) :: a
    real(8), intent (in) :: b
    code = sqrt(((a * a) - (b * b)))
end function

public static double code(double a, double b) {
	return Math.sqrt(((a * a) - (b * b)));
}

def code(a, b):
	return math.sqrt(((a * a) - (b * b)))

function code(a, b)
	return sqrt(Float64(Float64(a * a) - Float64(b * b)))
end

function tmp = code(a, b)
	tmp = sqrt(((a * a) - (b * b)));
end

code[a_, b_] := N[Sqrt[N[(N[(a * a), $MachinePrecision] - N[(b * b), $MachinePrecision]), $MachinePrecision]], $MachinePrecision]

\begin{array}{l}

\\
\sqrt{a \cdot a - b \cdot b}
\end{array}

Sampling outcomes in binary64 precision:

Local Percentage Accuracy vs ?

The average percentage accuracy by input value. Horizontal axis shows value of an input variable; the variable is choosen in the title. Vertical axis is accuracy; higher is better. Red represent the original program, while blue represents Herbie's suggestion. These can be toggled with buttons below the plot. The line is an average while dots represent individual samples.

Accuracy vs Speed?

Herbie found 2 alternatives:

AlternativeAccuracySpeedup
The accuracy (vertical axis) and speed (horizontal axis) of each alternatives. Up and to the right is better. The red square shows the initial program, and each blue circle shows an alternative.The line shows the best available speed-accuracy tradeoffs.

Initial Program: 52.8% accurate, 1.0× speedup?

\[\begin{array}{l} \\ \sqrt{a \cdot a - b \cdot b} \end{array} \]
(FPCore (a b) :precision binary64 (sqrt (- (* a a) (* b b))))
double code(double a, double b) {
	return sqrt(((a * a) - (b * b)));
}

real(8) function code(a, b)
    real(8), intent (in) :: a
    real(8), intent (in) :: b
    code = sqrt(((a * a) - (b * b)))
end function

public static double code(double a, double b) {
	return Math.sqrt(((a * a) - (b * b)));
}

def code(a, b):
	return math.sqrt(((a * a) - (b * b)))

function code(a, b)
	return sqrt(Float64(Float64(a * a) - Float64(b * b)))
end

function tmp = code(a, b)
	tmp = sqrt(((a * a) - (b * b)));
end

code[a_, b_] := N[Sqrt[N[(N[(a * a), $MachinePrecision] - N[(b * b), $MachinePrecision]), $MachinePrecision]], $MachinePrecision]

\begin{array}{l}

\\
\sqrt{a \cdot a - b \cdot b}
\end{array}

Alternative 1: 76.2% accurate, 1.0× speedup?

\[\begin{array}{l} \\ \begin{array}{l} \mathbf{if}\;a \leq -2 \cdot 10^{-270}:\\ \;\;\;\;\sqrt{\left(a + b\right) \cdot \left(a - b\right)}\\ \mathbf{else}:\\ \;\;\;\;a\\ \end{array} \end{array} \]
(FPCore (a b)
 :precision binary64
 (if (<= a -2e-270) (sqrt (* (+ a b) (- a b))) a))
double code(double a, double b) {
	double tmp;
	if (a <= -2e-270) {
		tmp = sqrt(((a + b) * (a - b)));
	} else {
		tmp = a;
	}
	return tmp;
}

real(8) function code(a, b)
    real(8), intent (in) :: a
    real(8), intent (in) :: b
    real(8) :: tmp
    if (a <= (-2d-270)) then
        tmp = sqrt(((a + b) * (a - b)))
    else
        tmp = a
    end if
    code = tmp
end function

public static double code(double a, double b) {
	double tmp;
	if (a <= -2e-270) {
		tmp = Math.sqrt(((a + b) * (a - b)));
	} else {
		tmp = a;
	}
	return tmp;
}

def code(a, b):
	tmp = 0
	if a <= -2e-270:
		tmp = math.sqrt(((a + b) * (a - b)))
	else:
		tmp = a
	return tmp

function code(a, b)
	tmp = 0.0
	if (a <= -2e-270)
		tmp = sqrt(Float64(Float64(a + b) * Float64(a - b)));
	else
		tmp = a;
	end
	return tmp
end

function tmp_2 = code(a, b)
	tmp = 0.0;
	if (a <= -2e-270)
		tmp = sqrt(((a + b) * (a - b)));
	else
		tmp = a;
	end
	tmp_2 = tmp;
end

code[a_, b_] := If[LessEqual[a, -2e-270], N[Sqrt[N[(N[(a + b), $MachinePrecision] * N[(a - b), $MachinePrecision]), $MachinePrecision]], $MachinePrecision], a]

\begin{array}{l}

\\
\begin{array}{l}
\mathbf{if}\;a \leq -2 \cdot 10^{-270}:\\
\;\;\;\;\sqrt{\left(a + b\right) \cdot \left(a - b\right)}\\

\mathbf{else}:\\
\;\;\;\;a\\


\end{array}
\end{array}
Derivation
  1. Split input into 2 regimes

  2. if a < -2.0000000000000001e-270

    1. Initial program 54.6%

      \[\sqrt{a \cdot a - b \cdot b} \]
    2. Step-by-step derivation

      1. difference-of-squares54.8%

        \[\leadsto \sqrt{\color{blue}{\left(a + b\right) \cdot \left(a - b\right)}} \]

    3. Simplified54.8%

      \[\leadsto \color{blue}{\sqrt{\left(a + b\right) \cdot \left(a - b\right)}} \]

    if -2.0000000000000001e-270 < a

    1. Initial program 52.4%

      \[\sqrt{a \cdot a - b \cdot b} \]
    2. Step-by-step derivation

      1. difference-of-squares52.9%

        \[\leadsto \sqrt{\color{blue}{\left(a + b\right) \cdot \left(a - b\right)}} \]

    3. Simplified52.9%

      \[\leadsto \color{blue}{\sqrt{\left(a + b\right) \cdot \left(a - b\right)}} \]

    4. Taylor expanded in a around inf 99.9%

      \[\leadsto \color{blue}{a} \]
  3. Recombined 2 regimes into one program.

  4. Final simplification78.3%

    \[\leadsto \begin{array}{l} \mathbf{if}\;a \leq -2 \cdot 10^{-270}:\\ \;\;\;\;\sqrt{\left(a + b\right) \cdot \left(a - b\right)}\\ \mathbf{else}:\\ \;\;\;\;a\\ \end{array} \]

Alternative 2: 50.2% accurate, 107.0× speedup?

\[\begin{array}{l} \\ a \end{array} \]
(FPCore (a b) :precision binary64 a)
double code(double a, double b) {
	return a;
}

real(8) function code(a, b)
    real(8), intent (in) :: a
    real(8), intent (in) :: b
    code = a
end function

public static double code(double a, double b) {
	return a;
}

def code(a, b):
	return a

function code(a, b)
	return a
end

function tmp = code(a, b)
	tmp = a;
end

code[a_, b_] := a

\begin{array}{l}

\\
a
\end{array}
Derivation

  1. Initial program 53.5%

    \[\sqrt{a \cdot a - b \cdot b} \]
  2. Step-by-step derivation

    1. difference-of-squares53.9%

      \[\leadsto \sqrt{\color{blue}{\left(a + b\right) \cdot \left(a - b\right)}} \]

  3. Simplified53.9%

    \[\leadsto \color{blue}{\sqrt{\left(a + b\right) \cdot \left(a - b\right)}} \]

  4. Taylor expanded in a around inf 52.5%

    \[\leadsto \color{blue}{a} \]

  5. Final simplification52.5%

    \[\leadsto a \]

Developer target: 99.2% accurate, 0.2× speedup?

\[\begin{array}{l} \\ \sqrt{\left|a\right| + \left|b\right|} \cdot \sqrt{\left|a\right| - \left|b\right|} \end{array} \]
(FPCore (a b)
 :precision binary64
 (* (sqrt (+ (fabs a) (fabs b))) (sqrt (- (fabs a) (fabs b)))))
double code(double a, double b) {
	return sqrt((fabs(a) + fabs(b))) * sqrt((fabs(a) - fabs(b)));
}

real(8) function code(a, b)
    real(8), intent (in) :: a
    real(8), intent (in) :: b
    code = sqrt((abs(a) + abs(b))) * sqrt((abs(a) - abs(b)))
end function

public static double code(double a, double b) {
	return Math.sqrt((Math.abs(a) + Math.abs(b))) * Math.sqrt((Math.abs(a) - Math.abs(b)));
}

def code(a, b):
	return math.sqrt((math.fabs(a) + math.fabs(b))) * math.sqrt((math.fabs(a) - math.fabs(b)))

function code(a, b)
	return Float64(sqrt(Float64(abs(a) + abs(b))) * sqrt(Float64(abs(a) - abs(b))))
end

function tmp = code(a, b)
	tmp = sqrt((abs(a) + abs(b))) * sqrt((abs(a) - abs(b)));
end

code[a_, b_] := N[(N[Sqrt[N[(N[Abs[a], $MachinePrecision] + N[Abs[b], $MachinePrecision]), $MachinePrecision]], $MachinePrecision] * N[Sqrt[N[(N[Abs[a], $MachinePrecision] - N[Abs[b], $MachinePrecision]), $MachinePrecision]], $MachinePrecision]), $MachinePrecision]

\begin{array}{l}

\\
\sqrt{\left|a\right| + \left|b\right|} \cdot \sqrt{\left|a\right| - \left|b\right|}
\end{array}

Reproduce

?
herbie shell --seed 2023192 
(FPCore (a b)
  :name "bug366, discussion (missed optimization)"
  :precision binary64

  :herbie-target
  (* (sqrt (+ (fabs a) (fabs b))) (sqrt (- (fabs a) (fabs b))))

  (sqrt (- (* a a) (* b b))))