After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 106442 - MMX code for paint_funcs problem in OpenBSD/i386
MMX code for paint_funcs problem in OpenBSD/i386
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: General
1.x
Other All
: Normal normal
: 2.0
Assigned To: GIMP Bugs
Daniel Egger
Depends on:
Blocks:
 
 
Reported: 2003-02-18 18:36 UTC by Jim Geovedi
Modified: 2003-07-05 17:28 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch-app_arch_i386_mmx_paint_funcs_mmx_S (2.65 KB, patch)
2003-02-18 18:39 UTC, Jim Geovedi 		none Details | Review

Description Jim Geovedi 2003-02-18 18:36:57 UTC 
While testing gimp-1.3.11 and gimp-1.3.12 on OpenBSD/i386 , the build
process give me error.

../../app/arch/i386/mmx/paint_funcs_mmx.S: Assembler messages:
../../app/arch/i386/mmx/paint_funcs_mmx.S:33: Error: Alignment too large:
15. assumed.
[deleted]
../../app/arch/i386/mmx/paint_funcs_mmx.S:1430: Error: Alignment too large:
15. assumed.
gmake[3]: *** [paint-funcs-simd.o] Error 1
gmake[3]: Leaving directory `/foo/bar/gimp-1.3.12/app/paint-funcs'

workaround:
I'm not really sure if this can be repeatable on other system, but by
replacing the ``.align 16'' into ``.align 15'', might solving the compile
problem in OpenBSD/i386.
Comment 1 Jim Geovedi 2003-02-18 18:39:01 UTC 
Created attachment 14420 [details] [review]
patch-app_arch_i386_mmx_paint_funcs_mmx_S
Comment 2 Sven Neumann 2003-02-18 18:48:55 UTC 
That code isn't even used because it is broken and now it starts to
give compilation errors :-(

We'd probably do best by removing the MMX code altogether as long as
noone stands up who is able and willing to maintain it.
Comment 3 Henrik Brix Andersen 2003-07-04 21:54:07 UTC 
Could be closed now since the old MMX code has been removed from CVS HEAD?