Ok, that's just bad. If ya gonna commit others stuff, at least tell the other folks that ya gonna do it ;) Nobody likes being in the dark...even mushrooms ;)
dh On 09/29/2011 11:29 PM, Enlightenment SVN wrote: > Log: > you know.... i was reviewing this patch this morning and i found > several problems with it... but SOMEONE... (lucas) committed it > without even so much as replying to the list saying he was going to... > :) > > > > Author: raster > Date: 2011-09-29 20:29:03 -0700 (Thu, 29 Sep 2011) > New Revision: 63705 > Trac: http://trac.enlightenment.org/e/changeset/63705 > > Modified: > trunk/evas/src/lib/engines/common/evas_cpu.c > trunk/evas/src/lib/engines/common/evas_op_blend_main_.c > trunk/evas/src/lib/include/evas_blend_ops.h > > Modified: trunk/evas/src/lib/engines/common/evas_cpu.c > =================================================================== > --- trunk/evas/src/lib/engines/common/evas_cpu.c 2011-09-30 02:37:12 UTC > (rev 63704) > +++ trunk/evas/src/lib/engines/common/evas_cpu.c 2011-09-30 03:29:03 UTC > (rev 63705) > @@ -68,9 +68,9 @@ > evas_common_cpu_sse3_test(void) > { > #ifdef BUILD_SSE3 > - int data[4]; > + int data[4]; > > - __m128i val = _mm_lddqu_si128((__m128i *)data); > + _mm_lddqu_si128((__m128i *)data); > #endif > } > > @@ -170,10 +170,10 @@ > cpu_feature_mask&= ~CPU_FEATURE_SSE; > #ifdef BUILD_SSE3 > cpu_feature_mask |= CPU_FEATURE_SSE3 * > - evas_common_cpu_feature_test(evas_common_cpu_sse3_test); > + evas_common_cpu_feature_test(evas_common_cpu_sse3_test); > evas_common_cpu_end_opt(); > if(getenv("EVAS_CPU_NO_SSE3")) > - cpu_feature_mask&= ~CPU_FEATURE_SSE3; > + cpu_feature_mask&= ~CPU_FEATURE_SSE3; > #endif /* BUILD_SSE3 */ > #endif /* BUILD_SSE */ > #endif /* BUILD_MMX */ > > Modified: trunk/evas/src/lib/engines/common/evas_op_blend_main_.c > =================================================================== > --- trunk/evas/src/lib/engines/common/evas_op_blend_main_.c 2011-09-30 > 02:37:12 UTC (rev 63704) > +++ trunk/evas/src/lib/engines/common/evas_op_blend_main_.c 2011-09-30 > 03:29:03 UTC (rev 63705) > @@ -84,6 +84,9 @@ > # include "./evas_op_blend/op_blend_mask_color_i386.c" > //# include "./evas_op_blend/op_blend_pixel_mask_color_i386.c" > > +#ifdef BUILD_SSE3 > +static __m128i A_MASK_SSE3; > +#endif > # include "./evas_op_blend/op_blend_pixel_sse3.c" > # include "./evas_op_blend/op_blend_color_sse3.c" > # include "./evas_op_blend/op_blend_pixel_color_sse3.c" > @@ -410,17 +413,17 @@ > memset(op_blend_rel_span_funcs, 0, sizeof(op_blend_rel_span_funcs)); > memset(op_blend_rel_pt_funcs, 0, sizeof(op_blend_rel_pt_funcs)); > #ifdef BUILD_SSE3 > - init_blend_rel_pixel_span_funcs_c(); > - init_blend_rel_pixel_color_span_funcs_c(); > - init_blend_rel_pixel_mask_span_funcs_c(); > - init_blend_rel_color_span_funcs_c(); > - init_blend_rel_mask_color_span_funcs_c(); > + init_blend_rel_pixel_span_funcs_sse3(); > + init_blend_rel_pixel_color_span_funcs_sse3(); > + init_blend_rel_pixel_mask_span_funcs_sse3(); > + init_blend_rel_color_span_funcs_sse3(); > + init_blend_rel_mask_color_span_funcs_sse3(); > > - init_blend_rel_pixel_pt_funcs_c(); > - init_blend_rel_pixel_color_pt_funcs_c(); > - init_blend_rel_pixel_mask_pt_funcs_c(); > - init_blend_rel_color_pt_funcs_c(); > - init_blend_rel_mask_color_pt_funcs_c(); > + init_blend_rel_pixel_pt_funcs_sse3(); > + init_blend_rel_pixel_color_pt_funcs_sse3(); > + init_blend_rel_pixel_mask_pt_funcs_sse3(); > + init_blend_rel_color_pt_funcs_sse3(); > + init_blend_rel_mask_color_pt_funcs_sse3(); > #endif > #ifdef BUILD_MMX > init_blend_rel_pixel_span_funcs_mmx(); > > Modified: trunk/evas/src/lib/include/evas_blend_ops.h > =================================================================== > --- trunk/evas/src/lib/include/evas_blend_ops.h 2011-09-30 02:37:12 UTC > (rev 63704) > +++ trunk/evas/src/lib/include/evas_blend_ops.h 2011-09-30 03:29:03 UTC > (rev 63705) > @@ -9,6 +9,15 @@ > #include<immintrin.h> > #endif > > +#ifndef always_inline > +#if defined(__GNUC__)&& (__GNUC__> 3 || __GNUC__ == 3&& __GNUC_MINOR__> > 0) > +# define always_inline __attribute__((always_inline)) inline > +#else > +# define always_inline inline > +#endif > +#endif > + > + > /* src pixel flags: */ > > /* pixels none */ > @@ -191,7 +200,7 @@ > static __m128i RB_MASK_SSE3; > static __m128i SYM4_MASK_SSE3; > static __m128i RGB_MASK_SSE3; > -static __m128i A_MASK_SSE3; > +//static __m128i A_MASK_SSE3; > > static __m128i ALPHA_SSE3; > > > > ------------------------------------------------------------------------------ > All of the data generated in your IT infrastructure is seriously valuable. > Why? It contains a definitive record of application performance, security > threats, fraudulent activity, and more. Splunk takes this data and makes > sense of it. IT sense. And common sense. > http://p.sf.net/sfu/splunk-d2dcopy2 > _______________________________________________ > enlightenment-svn mailing list > enlightenment-...@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-svn > ------------------------------------------------------------------------------ All of the data generated in your IT infrastructure is seriously valuable. Why? It contains a definitive record of application performance, security threats, fraudulent activity, and more. Splunk takes this data and makes sense of it. IT sense. And common sense. http://p.sf.net/sfu/splunk-d2dcopy2 _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel