gallivm: fix comment wrt srgb accuracy.
authorRoland Scheidegger <sroland@vmware.com>
Fri, 2 Aug 2013 22:24:29 +0000 (00:24 +0200)
committerRoland Scheidegger <sroland@vmware.com>
Thu, 8 Aug 2013 16:55:57 +0000 (18:55 +0200)
I think it's actually not good enough now...

src/gallium/auxiliary/gallivm/lp_bld_format_srgb.c

index 848d6f8121b3ed8b061d14384c510f59fa7cb067..2b1fe643849324b8d631904bb1321bb1ec1a4616 100644 (file)
@@ -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).