On Saturday, June 4, 2016, John Paul Adrian Glaubitz < glaub...@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 - glaub...@debian.org <javascript:;> > `. `' Freie Universitaet Berlin - glaub...@physik.fu-berlin.de > <javascript:;> > `- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913 > >