summaryrefslogtreecommitdiffstats
path: root/src/gallium/auxiliary/gallivm
diff options
context:
space:
mode:
authorRoland Scheidegger <[email protected]>2013-08-03 00:24:29 +0200
committerRoland Scheidegger <[email protected]>2013-08-08 18:55:57 +0200
commit2d9fea95e8627e7524bc4263bc6f9067fcf4db5e (patch)
tree8cdacdb0d94718dce3d9a867b19fcc4e8a1605e4 /src/gallium/auxiliary/gallivm
parentf9a4288bd282dfab97a40fc6dc9c046812e7d681 (diff)
gallivm: fix comment wrt srgb accuracy.
I think it's actually not good enough now...
Diffstat (limited to 'src/gallium/auxiliary/gallivm')
-rw-r--r--src/gallium/auxiliary/gallivm/lp_bld_format_srgb.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/src/gallium/auxiliary/gallivm/lp_bld_format_srgb.c b/src/gallium/auxiliary/gallivm/lp_bld_format_srgb.c
index 848d6f8121b..2b1fe643849 100644
--- a/src/gallium/auxiliary/gallivm/lp_bld_format_srgb.c
+++ b/src/gallium/auxiliary/gallivm/lp_bld_format_srgb.c
@@ -242,8 +242,10 @@ lp_build_linear_to_srgb(struct gallivm_state *gallivm,
* error metric if you'd want to tweak them, they also MUST fit with
* the crappy polynomial above for srgb->linear since it is required
* that each srgb value maps back to the same value).
- * This function has an error of max +-0.17 (and we'd only require +-0.6),
- * for the approximated srgb->linear values the error is naturally larger
+ * This function has an error of max +-0.17. Not sure this is actually
+ * enough, we require +-0.6 but that may include the +-0.5 from integer
+ * conversion. Seems to pass all relevant tests though...
+ * For the approximated srgb->linear values the error is naturally larger
* (+-0.42) but still accurate enough (required +-0.5 essentially).
* All in all (including min/max clamp, conversion) 15 instructions.
* FMA would help (minus 2 instructions).