Neon ColorMatrix and Sepia have overflow

watery01 / libyuv

Automatically exported from code.google.com/p/libyuv

0 stars 0 forks source link

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

Run Sepia on Neon.
Observe white becoming colorful.

Original issue reported on code.google.com by fbarch...@google.com on 1 Apr 2013 at 11:27

GoogleCodeExporter commented 9 years ago

r630 disables Neon.
Sepia is mostly ok but clamping early.  Patch forthcoming.
ColorMatrix overflows and needs more work.

Original comment by fbarch...@google.com on 2 Apr 2013 at 10:48

GoogleCodeExporter commented 9 years ago

r631 fixes Neon sepia.
ColorMatrix is using C for now.

Original comment by fbarch...@google.com on 2 Apr 2013 at 6:58

GoogleCodeExporter commented 9 years ago

Fixed in r644
TestARGBColorMatrix (3021 ms)
TestARGBSepia (1537 ms)

Was C
TestARGBColorMatrix (21301 ms)
TestARGBSepia (10070 ms)

Original comment by fbarch...@chromium.org on 5 Apr 2013 at 4:18