Bug 196188

Summary: graphics/ImageMagick: On i386 bogus -march=penryn is appended behind -march=core2
Product: Ports & Packages Reporter: Mikhail T. <freebsd-2024>
Component: Individual Port(s)Assignee: Koop Mast <kwm>
Status: Closed FIXED    
Severity: Affects Some People CC: miwi
Priority: --- Flags: bugzilla: maintainer-feedback? (kwm)
Version: Latest   
Hardware: i386   
OS: Any   

Description Mikhail T. 2014-12-22 09:21:04 UTC 
I think, this is an upstream bug -- and I reported it there:

http://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=26747&p=118227

But meanwhile something needs to be done about it -- the port was miscompiled here and crashing (from SIGILL) because it tried to use the SSE-instructions, which my CPU does not have.

My workaround was to run `make configure' and then edit the generated files to remove the bogus flag from CFLAGS. But I'm not sure, what other unasked-for values are added -- it may not be just the penryn...
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2014-12-22 09:21:04 UTC 
Auto-assigned to maintainer kwm@FreeBSD.org
Comment 2 Koop Mast freebsd_committer freebsd_triage 2014-12-22 11:01:20 UTC 
Do you have the SIMD option enabled by chance? If so can you try to disable that as a work around for now?
Comment 3 Mikhail T. 2014-12-22 15:15:37 UTC 
(In reply to Koop Mast from comment #2)
> Do you have the SIMD option enabled by chance? If so can you try to disable
> that as a work around for now?

Yes, indeed. Turning this option off fixes the problem. The description of the autofoo macro impementing the --with-gcc-arch feature says:

http://www.gnu.org/software/autoconf-archive/ax_gcc_archflag.html

>> The main emphasis here is on recent CPUs, on the principle that doing high-performance computing on old hardware is uncommon.

I'd recommend removing the SIMD-option from the port entirely -- FreeBSD already allows one to specify the CPUTYPE (in /etc/make.conf) and that, at least, will come with no surprises...

Thanks!
Comment 4 commit-hook freebsd_committer freebsd_triage 2014-12-30 21:48:48 UTC 
A commit references this bug:

Author: kwm
Date: Tue Dec 30 21:48:39 UTC 2014
New revision: 375859
URL: https://svnweb.freebsd.org/changeset/ports/375859

Log:
  Update ImageMagick to 6.9.0-2.

  Disable SIMD option, the --with-gcc-arch only takes recent cpus into account.
  And doesn't propperly check the host cpu for the flags. [1]

  PR:		196188 [1]
  Submitted by:	mi@ALDAN.algebra.com

Changes:
  head/graphics/ImageMagick/Makefile
  head/graphics/ImageMagick/distinfo
  head/graphics/ImageMagick/files/
Comment 5 Koop Mast freebsd_committer freebsd_triage 2014-12-30 22:13:01 UTC 
Keeping this open as a reminder for myself that I would like a proper SIMD option..