On Wed, Aug 10, 2011 at 4:04 PM, H.J. Lu <hjl.to...@gmail.com> wrote: > On Wed, Aug 10, 2011 at 6:46 AM, Richard Guenther > <richard.guent...@gmail.com> wrote: >> On Sat, Mar 29, 2008 at 10:11 PM, H.J. Lu <hjl.to...@gmail.com> wrote: >>> This patch restores proper checking the third argument on blendpd and >>> and blendps. It also adds 2 tests, including pblendw. Tested on >>> Linux/Intel64. OK to install? >> >> The gcc.target/i386/sse4_1-blendps-2.c test randomly fails because >> src3 is used uninitialized. >> > > SRC2 may be uninitialized. But I never saw random failures > since it checks if random value in SRC2 is properly blended.
No, src2 is initialized via init_blendps (src1.f, src2.f), src3 is uninitialized. I see random execute fails on Nehalem. I suppose we might optimize the uninitialized memory (it's probably committed to registers) based on the undefined behavior. Please avoid this by initializing src3 properly. Richard. > -- > H.J. >
