[Piglit] [PATCH v3 2/3] arb_shader_precision: add framework for calculating tolerances for complex functions
Ilia Mirkin
imirkin at alum.mit.edu
Mon Mar 9 13:02:46 PDT 2015
On Mon, Mar 9, 2015 at 3:51 PM, Micah Fedke <micah.fedke at collabora.co.uk> wrote:
> Still no idea what to do with a*b+c in complex functions - don't really want
> to evaluate all the permutations if at all possible.
Yeah, that stinks, but the reality is that _something_ needs to be
done. A bunch of tests on nvc0 will fail otherwise since we merge
a*b+c into fma opportunistically. This seems to be perfectly legal per
the spec.
I briefly glanced at your changes, and outside of some trivialities
they seem mostly fine. I'd really prefer it if you changed
+def _analyze_ref_fn(fn, args):
+ interval_results = fn(args)
to fn(*args). Then you avoid all the args[0]/etc insanity since the
arguments just come out as you'd expect.
But the a*b+c disaster needs some sort of resolution. I'm still at a
bit of a loss.
-ilia
More information about the Piglit
mailing list