More refactoring notes

Add refactoring notes

Mention utils.h in test/Makefile.am so that make dist will pick it up

Pointed out by Julien Cristau.

Move all the GTK+ test code into its own file, utils.c

Include string.h and fix warning in trap-test.c

Add solaris-hwcap.mapfile to EXTRA_DIST

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

Updated scaling test to support rgb565, source clipping, etc.

Now test provides better coverage for various image scaling
cases. Unused byte for x8r8g8b8 format is ignored. Running
the test program without any command line options provides
a PASS/FAIL verdict based on the precalculated crc32 value
for using pixman with all the fastpath functions disabled,
which should simplify testing for correcteness.

Add trap-crasher.c test program

Based off of Pavel Kankovsky's test case in bug 16560.

Enable fbCompositeSolidMask_nx8x8888neon

Enable fbCompositeSolidMask_nx8x0565neon

Enable fbCompositeSrc_8888x8x8888neon

Enable fbCompositeSrc_8888x8888neon

Enable fbCompositeSrc_x888x0565neon

Enable fbCompositeSrcAdd_8000x8000neon

Enable fbCompositeSrcAdd_8888x8x8neon

Add support for ARM NEON fast paths

Currently disabled

Add support for doing ARM simd detection on Windows

Add support for doing ARM simd detection on Windows

Post-release version bump

Pre-release version bump

Fix search and replace issue

Replace custom type ullong with standard uint64_t in pixman-mmx.c

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

Fix MMX & SSE intrinsics to work with Sun compilers & Solaris

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

Fix pixbuf_from_argb32() to take premultiplied alpha into account

Add a new trap-test test program.

Also some tweaks to the build system in the test directory to make it
build non-gtk+-using application when use of gtk+ is disabled.

Set srcRepeat = False when using fbCompositeSrcScaleNearest

NORMAL repeat was broken (the optimized function can handle repeat operation
itself and can be screwed up if 'pixman_walk_composite_region' tries to help it
by splitting the work into handling multiple separate areas).

Splitting work into handling different areas does not work right for the
transform case (and it is never used for generic path).  The point is that this
splitting only has full pixel precision at the moment, while correct blitting
needs to preserve some fractional part in calculations when moving from one
"tile" to another.

Image scaling regression test script

This test script can help in finding regressions in image scaling
fastpath implementations. It uses test program compiled with
and without fastpath code and can compare results of execution
for different pseudorandom compositing operations involving scaling.

Signed-off-by: Søren Sandmann Pedersen <sandmann@redhat.com>

Add support for BGRA and BGRx formats.

Check for failure when intersecting regions.

Need to check and report the failure of intersecting the rectangle with
the clip region during pixman_image_fill_rectangles().

Check for allocation errors during pixman_op()

Propagate the error returns from pixman_rect_alloc().

Remove stale comment

Add license and copyright holders to COPYING

Implement PIXMAN_REPEAT_REFLECT for images

Reinstate SrcScaledNearest optimization

Bump version number post-release

Bump version number pre release

Comment out SrcScaledNearest optimization that hasn't been tested much

Fix release targets to remove all hashfiles before generating tar balls

Add pixman-matrix.c to Makefile.win32

Conditionally include config.h in pixman-matrix.c to fix win32 build

Fix pixman-1-uninstalled.pc to point to the libtool library

Otherwise we fail to link when compiling cairo against the uninstalled
library.

Allocate initial array of RegionInfo on the stack.

The region validate() code is frequently called by cairo as it is used to
extract regions from the trapezoids for fast-paths through the drawing
code and also for fast-path clipping and the RegionInfo allocation (as
well as the pixman_rect_alloc during the final union) appears as a hot
spot on application memory profiles.

Don't treat PIXMAN_TYPE_YUY2 and PIXMAN_TYPE_YV12 as PIXMAN_FORMAT_COLOR.

Various pieces of code expect PIXMAN_FORMAT_COLOR (and its less cool older
brother, PICT_FORMAT_COLOR) formats to have ARGB bits, and the YUV formats do
not.

[arm-simd] Add a comment about aligning source and destination pointers.

Check alignment of 'src' pointer in optimized ARM routines

fbCompositeSrcAdd_8000x8000arm() tries to align 'dst' already but must check
'src' too.  Otherwise, the next 4-byte copy loop might access an odd 'src' address
causing an alignment trap.

Patch from Enrico Scholz

Merge commit 'origin/master'

Bump to 0.13.3 after 0.13.2 release

Bump version to 0.13.2 for release

Move matrix operations from X server to pixman

Signed-off-by: Keith Packard <keithp@keithp.com>

Optimize rectilinear nearest-neighbour scaling

Add a special case for a source transformation that is only a scale and
preserves rectangular pixels and doesn't rotate the image. Currently, only
SOURCE is special cased, however I plan to do more work in this area as needed.
The biggest advantage the specialization currently has is writing directly to
the destination surface instead of a temporary scanline buffer. However, it is
still pretty unoptimized but I want to keep things simple for now.

Unify scanline buffer types

Change the type of the stack based scanline buffer to uint8_t to match the rest
of the variables. Also premultiply the scanline buffer size by sizeof(uint32_t)
because the bpp can be either sizeof(uint32_t) or sizeof(uint64_t).

[arm-simd] Fix typo found by 王新拓

'and r7, %[upper_component_mask]' appears to by a short hand for
'and r7, %[upper_component_mask], %[upper_component_mask]'. Use
the explicit form to avoid any confusion.

Less fragile Linux altivec detection

Instead of using really fragile SIGILL trapping, use a more reliable
detection method by checking what the CPU really supports.

https://bugzilla.redhat.com/show_bug.cgi?id=472000
https://bugzilla.redhat.com/show_bug.cgi?id=451831

Make comments about PIXMAN_REFLECT more useful

Various formatting fixes, and a simplification of the adjusting code

Inline the fetchers

Use fetch_nearest() instead of having duplicated code

Do the fetch_bilinear inline rather than in separate functions

Make use of fetch_bilinear() in the various bilinear implementations

Add fetch_bilinear function

Add missing pixman-arm-simd.[ch] files.

Pointed out by Chris Ball and Adrian Bunk.

Inlucde inttypes.h on digital Tru64

Bug 18007, reported by Joonas Pihlaja.

Rename the current ARM code to ARM SIMD

This code is only for CPUs supporting the SIMD instructions, not for all ARM
CPUs.

I stumbled above the recent commit with the ARM SIMD code while preparing a
patch that models the patch from #13445 after the MMX and SSE2 cases:

The ARM SIMD option currently uses --disable-arm, although this code is only
for CPUs >= ARMv6. That's as if one would call the option to disable the SSE2
code --disable-x86.

This patch therefore renames the configure option and the function and file
names to arm-simd/arm_simd.

Define force_inline for compilers other than gcc & MS Visual C

Cleanup inline definitions

Replace all inline definitions with a common one in pixman-private.h. Also, add
'force_inline' and replace all existing uses of 'inline' as a forced inline
with 'force_inline'.

Remove unused AddMul_256 macro

unswitch red and blue

The previous code assumed a color format of ABGR when naming the
variables. The true color format is ARGB. This did not cause any bugs
because no functions rely on the order of colors so far. This patch
renames the variables just to avoid confusion.

update .gitignore

force alignment of arg pointer in potential SSE users

Fix for bug 17824

Move _mm_empty() to correct place

The "fbComposeSetupSSE2()" function is guarding most of its code
depending on the capabilities of the CPU, but unfortunately the call
to "_mm_empty()" is not part of this code path but executed
unconditionally.  This results in a "illegal instruction" crash on
non-MMX / non-SSE capable CPUs caused by the the "emms" instruction
(embedded in "_mm_empty()").

Fix bug 17729.

Update TODO

Make use of SSE2 blt/fill in more places

Merge tag 'pixman-0.12.0'

Conflicts:

configure.ac

Inline fetch functions in pixman-transformed.c

There was significant performance overhead associated with indirect
call to the fetch functions, so inline them.

Also, the old code assumed that if the number of rectangles in the
source clip was 1, then the clip was identical to the image
boundaries. Fix that by running a full region_contains_point()
whenever the source clip is different from pict->common.full_region.

Based on a patch from Antoine Azar.

Fix inner branch code.

The entire source must be 0 not just the alpha component.
Fix some comments too.

Post release version bump

Pre-release version bump

Add SRC x888x0565 C fast path

Add support for ARMv6 SIMD fastpaths.

Pre-release version bump

Don't include stdio.h

[sse2] Fix rounding bug in conversion from 565 to 8888

When converting from 565 to 8888, replicate the topmost bits instead
of appending zeros.

Fix for bug 17477.

over_2x128 was changing the alphaLo and alphaHi arguments, causing
stripes.

Update TODO

Extend clip-test to demonstrate a bug in source clipping

Fix bug in pixman_image_is_opaque()

Non-repeating gradient images would be reported as opaque. Also add
new test program to test source clipping.

Update RELEASING

Bump release

Check for __sun || __sun in pixman.h. Update TODO

Reported by Bernd Nies.

Use error instead of #error in a couple of other places

Minor portability fixes

Use AC_C_INLINE to figure out `inline'.
IRIX compiler does not exit with a non-zero exit status when it sees #error

Win32 build system fixes

Signed-off-by: Søren Sandmann Pedersen <sandmann@daimi.au.dk>

Make sure pixman-combine{32,64}.h are disted

Rename pixman-sse.h pixman-sse2.h

A few other renamings of SSE->SSE2

Be consistent in naming SSE2 related things SSE2

Remove use of MMX extensions.

They were never a big improvement anyway, and now that we have SSE2
support, they would only be useful on Pentium IIIs, and only if
explicitly enabled, which most distributions couldn't do anyway.

[sse2] Change pixman-sse to pass __mm128i args as pointers, so as not to confuse MSVC

Use hidden attribute for private functions when compiling with Sun Studio

https://bugs.freedesktop.org/show_bug.cgi?id=17183

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

Add sys/inttypes.h include for AIX