PATCH: R600 OpenCV fixes

Jan Vesely jan.vesely at rutgers.edu
Tue Jan 14 13:24:04 PST 2014


Hi,

On Mon, 2014-01-13 at 16:07 -0500, Jan Vesely wrote:
> Hi,
> 
> I tested these patches and they fix the piglit test I submitted, however
> they seem to introduce stack corruption. See
> https://bugs.freedesktop.org/show_bug.cgi?id=73571 for details.

I dug a bit deeper into this.

The third patch introduces (or uncovers) the stack corruption crash
mentioned in the above bug report (description).

The other patches are OK (run all gegl tests without crashing). The
second patch even fixes what I thought was a rounding issue in one of
the gegl tests. If it helps, all but the third one have my
Acked-by: Jan Vesely <jan.vesely at rutgers.edu>

I also ran piglit and there is no difference between master, master +
1,2,4,5. Adding patch 3 fixes the newly submitted 'program-scope-arrays'
test.


regards,
Jan


> 
> regards,
> Jan
> 
> On Thu, 2013-12-19 at 21:01 -0800, Tom Stellard wrote:
> > Hi,
> > 
> > There patches fix a few bugs uncovered by the OpenCV test suite.
> > 
> > Please Review.
> > -Tom
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 

-- 
Jan Vesely <jan.vesely at rutgers.edu>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140114/201f1a86/attachment.sig>


More information about the llvm-commits mailing list