[test] Exercise repeating patterns for composite.

[build] Add rule to generate asm for inspection.

[sse2] Don't emit prefetch 0 for an absent mask

[test] Add composite test from rendercheck

Iterate over all destination formats for dst, src and composite and
compare the result of all oprators with a selection of colours.

build: Suppress verbose compile lines

Compile warnings are being lost in the sea of noise. Automake-1.11 finally
introduced AM_SILENT_RULES to suppress the echoing of the compile line for
every object. Enable this to bring sanity to the pixman build.

Merge branch '0.16'

Conflicts:
configure.ac
pixman/pixman-sse2.c

Remove duplicated declaration

The pixman_tranform_pixman_f_transform() declaration is repeated 4 lines
down.

Enable component alpha on solid masks.

[sse2] Bit-reversing typo: src != dst

Fix off-by-one error in source_image_needs_out_of_bounds_workaround()

If extents->x2/y2 are equal to image->width/height, then the clip is
still inside the drawable, so no workaround is necessary.

Remove unused generated libcomp.pc #23801

Change CFLAGS order for PPC and ARM configure tests

CFLAGS are always appended to the end of gcc options when compiling
sources in autotools based projects. Configure tests should do the
same. Otherwise build fails on PPC when using CFLAGS="-O2 -mno-altivec"
for example. Similar problem affects ARM.

ARM: Remove fallback to ARMv6 implementation from NEON delegate chain

This can help to fix build problems with '-mthumb' gcc option in CFLAGS.
ARMv6 optimized code can't be compiled for thumb (because of its inline
assembly) and gets automatically disabled in configure. Reference
to it from NEON optimized code resulted in linking problems.

Every ARMv6 optimized fast path function also has a better NEON
counterpart, so there is no need to fallback to ARMv6. Shorter
delegate chain should additionally result in a bit better performance.

Change CFLAGS order for PPC and ARM configure tests

CFLAGS are always appended to the end of gcc options when compiling
sources in autotools based projects. Configure tests should do the
same. Otherwise build fails on PPC when using CFLAGS="-O2 -mno-altivec"
for example. Similar problem affects ARM.

ARM: Remove fallback to ARMv6 implementation from NEON delegate chain

This can help to fix build problems with '-mthumb' gcc option in CFLAGS.
ARMv6 optimized code can't be compiled for thumb (because of its inline
assembly) and gets automatically disabled in configure. Reference
to it from NEON optimized code resulted in linking problems.

Every ARMv6 optimized fast path function also has a better NEON
counterpart, so there is no need to fallback to ARMv6. Shorter
delegate chain should additionally result in a bit better performance.

Default to optimised builds when using a Sun Studio compiler.

Autoconf's AC_PROG_CC sets the default CFLAGS to -O2 -g for
gcc and -g for every other compiler.  This patch defaults
CFLAGS to the equivalent -O -g when we're using Sun Studio's cc
if the user or site admin hasn't already set CFLAGS.

Work around a Sun Studio 12 code generation bug involving _mm_set_epi32().

Calling a static function wrapper around _mm_set_epi32() when not
using optimisation causes Sun Studio 12's cc to emit a spurious
floating point load which confuses the assembler.  Using a macro wrapper
rather than a function steps around the problem.

Work around differing _mm_prefetch() prototypes on Solaris.

Sun Studio 12 expects the address to prefetch to be
a const char pointer rather than a __m128i pointer or
void pointer.

Default to optimised builds when using a Sun Studio compiler.

Autoconf's AC_PROG_CC sets the default CFLAGS to -O2 -g for
gcc and -g for every other compiler.  This patch defaults
CFLAGS to the equivalent -O -g when we're using Sun Studio's cc
if the user or site admin hasn't already set CFLAGS.

Work around a Sun Studio 12 code generation bug involving _mm_set_epi32().

Calling a static function wrapper around _mm_set_epi32() when not
using optimisation causes Sun Studio 12's cc to emit a spurious
floating point load which confuses the assembler.  Using a macro wrapper
rather than a function steps around the problem.

Work around differing _mm_prefetch() prototypes on Solaris.

Sun Studio 12 expects the address to prefetch to be
a const char pointer rather than a __m128i pointer or
void pointer.

ARM: workaround for gcc bug in vshll_n_u8 intrinsic

Some versions of gcc (cs2009q1, 4.4.1) incorrectly reject
shift operand having value >= 8, claiming that it is out of
range. So inline assembly is used as a workaround.

ARM: workaround for gcc bug in vshll_n_u8 intrinsic

Some versions of gcc (cs2009q1, 4.4.1) incorrectly reject
shift operand having value >= 8, claiming that it is out of
range. So inline assembly is used as a workaround.

Enable the x888_8_8888 sse2 fast path.

Set version number to 0.16.1

Add CPU detection for VC++ x64

VC++ x64 has no inline assembler and x64 mode supports SSE2.
So, it is unnecessary to call cpuid.

Add CPU detection for VC++ x64

VC++ x64 has no inline assembler and x64 mode supports SSE2.
So, it is unnecessary to call cpuid.

Change names of add_8888_8_8 fast paths to add_n_8_8

The source is solid in those.

Post-release version bump

Pre-release version bump

_pixman_run_fast_path: typo

This is one example of a compiler warning that was lost amit the build
noise.

The error here is that in a list of required conditions we used ';'
instead of '&&' with the result of continuing to use the fast-path
even if we had a wide mask.

Another error is that it was testing src, not mask as it should.

Remove spurious spaces in pixman-x64-mmx-emulation.h

Check if we have posix_memalign() in configure.ac. [23260, 23261]

Fall back to malloc() in blitters-test.c if we don't.

ARM: a fix to pass blitters-test for 'neon_composite_over_n_8_0565'

Inline assembly for handling <8 pixels width did not pass blitters-test.
Fortunately gcc has no problems compiling alternative implementation
which is using RVCT style intrinsics, so it can be used instead.

Post-release version bump

Pre-release version-bump

Merge branch 'blitter-test'

Fix x/y mixup in bits_image_fetch_pixel_convolution()

Bug 23224, reported by Michel Dänzer.

Update CRC value in blitters-test.

At this point, the SIMD, SSE2, MMX and general implementations all
agree.

Various formatting fixes

Add the ability to print intermediate CRC values

Reenable commented-out tests in blitter-test.

The crashes and valgrind issues are all fixed at this point.

One more update to blitters-test - use aligned memory
allocations in order in order to make reproducibility
of alignment sensitive bugs more deterministic
Also testing of masks is reenabled

HACK: updated test to better cover new neon optimizations

Test program for stressing the use of different formats and operators

The code and overall method is mostly based on scaling-test. This one
focuses on trying to stress as many different color formats and types
of composition operations as possible.

This is an initial implementation which may need more tuning. Also
not all color format and operator combinations are actually used.

When cpu specific optimizations are disabled, this test provides
identical deterministic results on x86, PPC and ARM.

Script blitters-test-bisect.rb now works in non-stop mode, until
it finds any problem. This allows to run it for example overnight
in order to test a lot more variants of pixman calls and increase
chances of detecting problems in pixman. Just like with scaling-test,
running blitters-test binary alone with no command line arguments
runs a small predefined number of tests and compares checksum
with a reference value for quick verification.

Delete commented out code in pixman-vmx.c

Misc formatting fixes for pixman-vmx.c

In vmx_combine_atop_reverse_ca() extract alpha after inversing

Really fix vmx_combine_over_reverse_ca()

The inverse destination alpha is just one component, not four.

Fix vmx_combine_out_reverse_ca()

The source alpha is just one component, not four.

Fix vmx_over_reverse_ca()

Destination alpha must be extracted after inversing, otherwise we end
up with 0xFFs in the rgb channels.

Multiply with the alpha of dest, not inverse alpha

Fix vmx_combine_vmx_atop_ca()

It didn't compute the mask correct before.

Fix vmx_combine_over_ca().

In the non-vector code, the mask needs to be multiplied with source
alpha.

In vmx_combine_out_ca() multiply with the alpha of the negated vdest.

Fix vmx_combine_out_ca()

It should multiply with just the destination alpha channel, not all
four channels.

Do the full four-component IN computation in vmx_combine_in_ca().

Fix bug in vmx_combine_xor_ca()

The destination needs to be inverted before the alpha channel is
extracted; otherwise, the RGB channels of da will be 0xff.

Make pix_multiply bit-exact

Change the SSE2 versions of pix_add_multiply() to produce bit-exact results.

Fix a couple of alpha==0 vs src==0 issues in pixman-sse2.c

Rename mmx_composite_add_8888_8_8() to mmx_composite_add_n_8_8().

Fix a couple more alpha==0 vs src==0 bugs in pixman-mmx.c

Make pix_add_mul() in pixman-mmx.c produce exact results.

Previously this routine would compute (x * a + y * b) / 255. Now it
computes (x * a) / 255 + (y * b) / 255, so that the results are
bitwise equivalent to the non-mmx versions.

Rewrite the two-component arithmetic macros.

Previously they were not bit-for-bit equivalent to the one-component
versions. The new code is also simpler and easier to read because it
factors out some common sub-macros.

The x * a + y * b macro now only uses four multiplications - the
previous version used eight.

Fix a bunch of srca == 0 checks that should be src == 0 in pixman-mmx.c

Don't run fast paths if the format requires wide compositing.

This could happen because the wide formats would still be considered
solid if the image was 1x1 and repeating.

Fix bug in combine_mask_alpha_ca()

If the mask was 0xffffffff, the source would end up being shifted
twice by A_SHIFT.

Fix another case of changing the solid source.

This time in fast_path_composite_n_8888_8888().

Fix incorrect optimization in combine_over_ca().

Previously the code assumed that an alpha of 0 meant that no change
would take place. This is incorrect because an alpha of 0 can happen
as the result of the source having alpha=0, but rgb != 0.

Don't change the constant source in fast_composite_over_n_8888_0565.

Fix bugs in combine_over_reverse_ca().

The computation cannot be optimized away when alpha is 0 because that
can happen when the source has alpha zero and rgb non-zero.

Add a dirty bit to the image struct, and validate before using the image.

This cuts down the number of property_changed calls significantly.

Add sse2 version of add_n_8888_8888()

Add a fast path for the add_n_8888_8888() operation.

It shows up on gnome-terminal traces.

Move bounds checks for REPEAT_NONE to get_pixel()

On a P4, this is a large speedup for the swfdec-fill-rate-2xaa trace:

After:

[ # ]  backend                         test   min(s) median(s) stddev. count
[  0]    image        swfdec-fill-rate-2xaa   33.061   33.061   0.00%    1/1

Before:

[ # ]  backend                         test   min(s) median(s) stddev. count
[  0]    image        swfdec-fill-rate-2xaa   40.342   40.342   0.00%    1/1

Pixman 0.14.0 produces this:

[ # ]  backend                         test   min(s) median(s) stddev. count
[  0]    image        swfdec-fill-rate-2xaa   36.896   36.896   0.00%    1/1

Remove leftover 0xffffffff in repeat()

Remove unused function

Misc formatting

Change all the fetch_pixels() functions to only fetch one pixel.

Add fetch_pixel_raw_32 and fetch_pixel_32 virtual functions.

By default both are intialized to bits_image_fetch_pixel_raw(), but if
there is an alpha map, then fetch_pixel_32() is set to
bits_image_fetch_pixel_alpha().

Various renamings and clean-ups

Change bits_image_fetch_alpha_pixels() to fetch just one pixel.

Change bits_image_fetch_pixels_convolution() to fetch just one pixel.

Change bits_image_fetch_bilinear_pixels() to fetch one pixel at a time.

Make the repeat routine work on only one coordinate at a time.

Make bits_image_fetch_nearest() return one pixel.

Previously it would work on a buffer of coordinates.

Change bits_image_fetch_transformed() to work one pixel at a time.

Previously, it would generate a buffer of coordinates, then pass that
off to a pixel fetcher, but this caused a large performance regression
with the swfdec-fill-rate-2xfsaa cairo trace.

This is the first step towards fixing that.

Only define PIXMAN_TIMERS if timers are actually enabled [bug 23169]

Various updates to the CODING_STYLE document

Add a CODING_STYLE document based on the one from cairo.

Remove a couple of unused variables

Rename source_pict_class_t to source_image_class_t

Replace a bunch of 'pict's with 'image'

Explain how we can simplify the radial gradient computation

Soeren rightfully complained that I had removed all the comments from
André's patch, most importantly that explain why the transformation is
valid. So add a few details to show that B varies linearly across the
scanline and how we can therefore reduce the per-pixel cost of evaluating
B.

Fix inversion of radial gradients when r2 > r1

Fixes: Bug 22908 -- Invalid output of radial gradient
http://bugs.freedesktop.org/show_bug.cgi?id=22908

We also include a modified patch by André Tupinambá <andrelrt@gmail.com>,
to pull constant expressions out of the inner radial gradient walker.

Don't warn for empty rectangles, only degenerate ones

Log errors for invalid rectangles passed to region code

Simplify code that logs errors

Make the text when reporting a broken region more useful