Re: help building cairo on windows

	MSVC does not support variables defined in the middle of the block, 
they have to be at the beginning. I guess since gcc/clang support that 
from a long time it's no issue for the rest of the platforms.

On 9/10/2012 1:40 PM, Cosmin Apreutesei wrote:
> I have no idea what you did (apart from the first line) but it worked. Thanks!
>
> On Mon, Sep 10, 2012 at 9:17 PM, Dimiter 'malkia' Stanev
> <[email protected]> wrote:
>> On 9/10/2012 7:04 AM, Cosmin Apreutesei wrote:
>>>
>>> Hi,
>>>
>>> I have to build cairo on windows because I want to use the recording
>>> surface and I can't find binaries above 1.10 anywhere on the net (till
>>> now I have used the binaries from GTK but these are old and the
>>> recording surface doesn't work with those).
>>>
>>> Trying to build pixman with:
>>>
>>>> make CFG=debug -f Makefile.win32
>>>
>>>
>>> gives me:
>>>
>>> pixman-mmx.c(545) : error C2143: syntax error : missing ';' before 'type'
>>>
>>> and then 100 syntax errors like that (other c files compile ok until
>>> the mmx one).
>>>
>>> The sources were pulled today from git. The environment is VSE2008.
>>> make is 3.81 from mingw (I also had to throw in pkg-config binaries +
>>> dependencies from GTK).
>>>
>>> I also tried to build the latest pixman (0.26.2) with the same command
>>> but it failed with:
>>>
>>> Entering directory `/x/work/pixman-0.26.2/test'
>>> Makefile.win32: No such file or directory
>>>
>>> I also tried building with mozilla-build from the mingw bash prompt,
>>> with the same results.
>>>
>>>
>>> So what's the best (preferably from cmdline) way to build pixman and
>>> cairo under windows? And which is the best version? The release or the
>>> git one?
>>>
>>> Btw, if anyone has an up-to-date cairo.dll that would work for me too.
>>>
>>
>> Hi Cosmin,
>>
>> With this patch you should be able to compile.
>>
>> Thanks,
>> Dimiter 'malkia' Stanev
>>
>> pixman-mmx.diff
>>
>> diff --git a/pixman/pixman-mmx.c b/pixman/pixman-mmx.c
>> index 74a5e87..9e597db 100644
>> --- a/pixman/pixman-mmx.c
>> +++ b/pixman/pixman-mmx.c
>> @@ -541,7 +541,7 @@ expand565 (__m64 pixel, int pos)
>>   static force_inline void
>>   expand_4xpacked565 (__m64 vin, __m64 *vout0, __m64 *vout1, int full_alpha)
>>   {
>> -    __m64 t0, t1, alpha = _mm_setzero_si64 ();;
>> +    __m64 t0, t1, alpha = _mm_setzero_si64 ();
>>       __m64 r = _mm_and_si64 (vin, MC (expand_565_r));
>>       __m64 g = _mm_and_si64 (vin, MC (expand_565_g));
>>       __m64 b = _mm_and_si64 (vin, MC (expand_565_b));
>> @@ -1902,22 +1902,22 @@ mmx_composite_over_8888_0565
>> (pixman_implementation_t *imp,
>>          while (w >= 4)
>>          {
>>              __m64 vdest = *(__m64 *)dst;
>> -           __m64 v0, v1, v2, v3;
>> +           __m64 v0, v1, v2, v3, vsrc0, vsrc1, vsrc2, vsrc3;
>>
>>              expand_4x565 (vdest, &v0, &v1, &v2, &v3, 0);
>>
>> -           __m64 vsrc0 = load8888 ((src + 0));
>> -           __m64 vsrc1 = load8888 ((src + 1));
>> -           __m64 vsrc2 = load8888 ((src + 2));
>> -           __m64 vsrc3 = load8888 ((src + 3));
>> -
>> +           vsrc0 = load8888 ((src + 0));
>> +           vsrc1 = load8888 ((src + 1));
>> +           vsrc2 = load8888 ((src + 2));
>> +           vsrc3 = load8888 ((src + 3));
>> +
>>              v0 = over (vsrc0, expand_alpha (vsrc0), v0);
>>              v1 = over (vsrc1, expand_alpha (vsrc1), v1);
>>              v2 = over (vsrc2, expand_alpha (vsrc2), v2);
>>              v3 = over (vsrc3, expand_alpha (vsrc3), v3);
>> -
>> +
>>              *(__m64 *)dst = pack_4x565 (v0, v1, v2, v3);
>> -
>> +
>>              w -= 4;
>>              dst += 4;
>>              src += 4;
>> @@ -2454,22 +2454,22 @@ mmx_composite_over_n_8_0565 (pixman_implementation_t
>> *imp,
>>              else if (m0 | m1 | m2 | m3)
>>              {
>>                  __m64 vdest = *(__m64 *)dst;
>> -               __m64 v0, v1, v2, v3;
>> +               __m64 v0, v1, v2, v3, vm0, vm1, vm2, vm3;
>>
>>                  expand_4x565 (vdest, &v0, &v1, &v2, &v3, 0);
>>
>> -               __m64 vm0 = to_m64 (m0);
>> +               vm0 = to_m64 (m0);
>>                  v0 = in_over (vsrc, vsrca, expand_alpha_rev (vm0), v0);
>> -
>> -               __m64 vm1 = to_m64 (m1);
>> +
>> +               vm1 = to_m64 (m1);
>>                  v1 = in_over (vsrc, vsrca, expand_alpha_rev (vm1), v1);
>> -
>> -               __m64 vm2 = to_m64 (m2);
>> +
>> +               vm2 = to_m64 (m2);
>>                  v2 = in_over (vsrc, vsrca, expand_alpha_rev (vm2), v2);
>> -
>> -               __m64 vm3 = to_m64 (m3);
>> +
>> +               vm3 = to_m64 (m3);
>>                  v3 = in_over (vsrc, vsrca, expand_alpha_rev (vm3), v3);
>> -
>> +
>>                  *(__m64 *)dst = pack_4x565 (v0, v1, v2, v3);;
>>              }
>>
>> @@ -3545,32 +3545,35 @@ mmx_composite_over_reverse_n_8888
>> (pixman_implementation_t *imp,
>>
>>   #define BILINEAR_INTERPOLATE_ONE_PIXEL(pix)
>> \
>>   do {
>> \
>> +    __m64 t_hi, t_lo, b_hi, b_lo, hi, lo;          \
>>       /* fetch 2x2 pixel block into 2 mmx registers */
>> \
>>       __m64 t = ldq_u ((__m64 *)&src_top [pixman_fixed_to_int (vx)]);
>> \
>>       __m64 b = ldq_u ((__m64 *)&src_bottom [pixman_fixed_to_int (vx)]);
>> \
>>       vx += unit_x;
>> \
>>       /* vertical interpolation */
>> \
>> -    __m64 t_hi = _mm_mullo_pi16 (_mm_unpackhi_pi8 (t, mm_zero), mm_wt);
>> \
>> -    __m64 t_lo = _mm_mullo_pi16 (_mm_unpacklo_pi8 (t, mm_zero), mm_wt);
>> \
>> -    __m64 b_hi = _mm_mullo_pi16 (_mm_unpackhi_pi8 (b, mm_zero), mm_wb);
>> \
>> -    __m64 b_lo = _mm_mullo_pi16 (_mm_unpacklo_pi8 (b, mm_zero), mm_wb);
>> \
>> -    __m64 hi = _mm_add_pi16 (t_hi, b_hi);
>> \
>> -    __m64 lo = _mm_add_pi16 (t_lo, b_lo);
>> \
>> +    t_hi = _mm_mullo_pi16 (_mm_unpackhi_pi8 (t, mm_zero), mm_wt);
>> \
>> +    t_lo = _mm_mullo_pi16 (_mm_unpacklo_pi8 (t, mm_zero), mm_wt);
>> \
>> +    b_hi = _mm_mullo_pi16 (_mm_unpackhi_pi8 (b, mm_zero), mm_wb);
>> \
>> +    b_lo = _mm_mullo_pi16 (_mm_unpacklo_pi8 (b, mm_zero), mm_wb);
>> \
>> +    hi = _mm_add_pi16 (t_hi, b_hi);
>> \
>> +    lo = _mm_add_pi16 (t_lo, b_lo);
>> \
>>       if (BILINEAR_INTERPOLATION_BITS < 8)
>> \
>>       {
>> \
>> +        __m64 p, q;          \
>>          /* calculate horizontal weights */
>> \
>>          __m64 mm_wh = _mm_add_pi16 (mm_addc7, _mm_xor_si64 (mm_xorc7,
>> \
>>                            _mm_srli_pi16 (mm_x,
>> \
>>                                           16 -
>> BILINEAR_INTERPOLATION_BITS)));   \
>>          mm_x = _mm_add_pi16 (mm_x, mm_ux);
>> \
>>          /* horizontal interpolation */
>> \
>> -       __m64 p = _mm_unpacklo_pi16 (lo, hi);
>> \
>> -       __m64 q = _mm_unpackhi_pi16 (lo, hi);
>> \
>> +        p = _mm_unpacklo_pi16 (lo, hi);
>> \
>> +       q = _mm_unpackhi_pi16 (lo, hi);
>> \
>>          lo = _mm_madd_pi16 (p, mm_wh);
>> \
>>          hi = _mm_madd_pi16 (q, mm_wh);
>> \
>>       }
>> \
>>       else
>> \
>>       {
>> \
>> +        __m64 mm_lo_lo, mm_lo_hi, mm_hi_lo, mm_hi_hi;
>> \
>>          /* calculate horizontal weights */
>> \
>>          __m64 mm_wh_lo = _mm_sub_pi16 (mm_BSHIFT, _mm_srli_pi16 (mm_x,
>> \
>>                                          16 - BILINEAR_INTERPOLATION_BITS));
>> \
>> @@ -3578,10 +3581,10 @@ do {
>> \
>>                                          16 - BILINEAR_INTERPOLATION_BITS);
>> \
>>          mm_x = _mm_add_pi16 (mm_x, mm_ux);
>> \
>>          /* horizontal interpolation */
>> \
>> -       __m64 mm_lo_lo = _mm_mullo_pi16 (lo, mm_wh_lo);
>> \
>> -       __m64 mm_lo_hi = _mm_mullo_pi16 (hi, mm_wh_hi);
>> \
>> -       __m64 mm_hi_lo = _mm_mulhi_pu16 (lo, mm_wh_lo);
>> \
>> -       __m64 mm_hi_hi = _mm_mulhi_pu16 (hi, mm_wh_hi);
>> \
>> +       mm_lo_lo = _mm_mullo_pi16 (lo, mm_wh_lo);
>> \
>> +       mm_lo_hi = _mm_mullo_pi16 (hi, mm_wh_hi);
>> \
>> +       mm_hi_lo = _mm_mulhi_pu16 (lo, mm_wh_lo);
>> \
>> +       mm_hi_hi = _mm_mulhi_pu16 (hi, mm_wh_hi);
>> \
>>          lo = _mm_add_pi32 (_mm_unpacklo_pi16 (mm_lo_lo, mm_hi_lo),
>> \
>>                             _mm_unpacklo_pi16 (mm_lo_hi, mm_hi_hi));
>> \
>>          hi = _mm_add_pi32 (_mm_unpackhi_pi16 (mm_lo_lo, mm_hi_lo),
>> \
>>
>> --
>> cairo mailing list
>> [email protected]
>> http://lists.cairographics.org/mailman/listinfo/cairo
-- 
cairo mailing list
[email protected]
http://lists.cairographics.org/mailman/listinfo/cairo