Remove unused color_table and color_table_size fields

Remove BOUNDS() macro.

It was bounding the clip region to INT16_MIN, INT16_MAX, but this was
a relic from the X server. We don't need it since we are already
restricting the clip region to the geometry of the destination.

--enable-maintainer-mode is gone from configure, so remove it

Add default cases for all switch statements

Fixes compilation with -Wswitch-default. Compilation with -Wswitch-enums
works fine as is.

Fix compile warnings

ARM: Removal of unused/broken NEON code

Fix double semicolon; pointed out by Travis Griggs

Fix build with Visual Studio 2008

moved __m64 ms declaration in sse2_composite_over_x888_8_8888 to top
of function so it compiles with visual studio 2008

Fix composite on big-endian systems.

Data narrower than 32bpp is padded to an unsigned long and on
big-endian systems this shifts the value by the padding bits.

Fix fetch-test for big-endian systems.

Data narrower than 32bpp should be stored in the correct
endian. Reported by Andrea Canciani.

Add missing break in composite.c

pixman: Update .gitignore

Generalize to catch all .pc files. Add more tests.

Signed-off-by: Guillem Jover <guillem@hadrons.org>

In the compositing test, Don't try to use component alpha with solid fills.

It's not supported yet.

Update CRC value in blitters-test for the new bug fixes

Fix bug in blitters-test with BGRA formats.

When masking out the x bits, blitter-test would make the incorrect
assumption that the they were always in the topmost position. This is
not correct for formats of type PIXMAN_TYPE_BGRA.

Fix bugs in fetch_*_b2g3r3().

The red channel should only be shifted five positions, not six.

Fix bugs in a1b2g1r1.

The first bug is that it is treating the input as if it were a1r1g1b1;
the second one is that the red channel should only be shifted two
bits, not three.

Fix shift bug in fetch_scanline/pixel_a2b2g2r2()

0x30 * 0x55 is 0xff0, so the red channel should be shifted four bits,
not six.

Fix four bit formats.

The original Render code used to index pixels with their position in
bits in the image. When the scanline code was introduced pixels were
indexed in bytes, but the FETCH/STORE_4/8 macros still assumed bits.

This commit fixes that by making the FETCH/STORE_4 macros first
convert the index to bit position.

Hide PIXMAN_OP_NONE and PIXMAN_N_OPERATORS behind PIXMAN_INTERNAL_API.

These cannot sanely be used by applications since they may change in
new versions.

Add a few notes about testing to TODO

Fix alpha handling for 10 bpc formats.

These generally extracted the 2 bits of alpha, then shifted them 62
bits and replicated across 16 bits. Then they were shifted another 48
bits, making the resulting alpha channel 0.

Return result from pixman_image_set_transform().

Previously it would always return TRUE, even when malloc() had failed.

Revert "Enable component alpha on solid masks."

For consistency we will probably want to allow component alpha to be
set on all masks at some point, but this commit only enabled it for
solid images.

This reverts commit 29e22cf38e8abc54b9dddbdeb3909d02866a82a0.

[Makefile] Set the SIMD specific CFLAGS for inspecting asm.

Remove optimization for 0xffffffff and 0xff the add_n_8888_8888_ca fast path

This is an ADD operation, not an OVER. Fixes bug 23934, reported by
Siarhei Siamashka.

Don't prefetch from NULL in the SSE2 fast paths.

On an Athlon64 box prefetch from NULL slows down
the rgba OVER rgba fast for predominantly solid sources
by up to 3.5x in the one-rounded-rectangle test case
when run using a tiling polygon renderer.  This patch
conditionalises the prefetches of the mask everywhere
where the mask pointer may be NULL in a fast path.

Reformat test/composite.c to follow the standard coding style.

[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.