Re: Bug#806208: src:gegl: FTBFS on sparc64, tools/introspect segfaults
On Saturday, June 4, 2016, John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de> wrote:
On 06/03/2016 11:49 PM, John Paul Adrian Glaubitz wrote:
> So, maybe we're lucky and just the above patch will be enough :).
>
> Will build gegl now to verify this.
And, indeed, gegl now builds fine on sparc64 [1].
Note the comments in the upstream bug tracker though [2] where it
is suggested to use the following code instead:
dst[0] = src[0];
dst[1] = src[1];
dst[2] = src[2];
dst[3] = 255;
I have not verified that yet, but it seems this should actually work.
Yeah, the memcpy with a word will probably be wrong due to endian issues when it is stored because dst[3] would be written in the first byte, but they are assuming that dst[3] is the last byte, given that they do OR with 0xff << 24, i.e. filling dst[3] = 0xff.
Bonus points if you find a way to do this with VIS intrinsics! ;)
Adrian
> [1] https://buildd.debian.org/status/fetch.php?pkg=gegl&arch=sparc64&ver=0.3.6-4&stamp=1465032322
> [2] https://bugzilla.gnome.org/show_bug.cgi?id=767219
--
.''`. John Paul Adrian Glaubitz
: :' : Debian Developer - glaubitz@debian.org
`. `' Freie Universitaet Berlin - glaubitz@physik.fu-berlin.de
`- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
Reply to: