Bug 215402 - multimedia/ffmpeg: only use vendor optimizations, dropping -ffast-math
Summary: multimedia/ffmpeg: only use vendor optimizations, dropping -ffast-math
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Thomas Zander
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-12-19 03:37 UTC by Jan Beich
Modified: 2017-01-01 18:57 UTC (History)
1 user (show)

See Also:
riggs: maintainer-feedback+


Attachments
v0 (2.33 KB, patch)
2016-12-19 03:37 UTC, Jan Beich
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Beich freebsd_committer freebsd_triage 2016-12-19 03:37:07 UTC
Created attachment 178083 [details]
v0

configure has --enable-optimizations (default) which includes -O3 -fomit-frame-pointer. Together with unconditional stripping it breaks unwinding as used by pmcstat/dtrace or -fsanitize=address. Rather than add PROFILE option let's use OPTIMIZED_CFLAGS to control vendor optimizations. But doing so induces POLA as -ffast-math from ports r69328 would contaminate defaults or left out.

In my opinion, extra optimization flags should not leave make.conf. Vendor doesn't test those, so having in port adds to maintenance to check with clang/gcc or on different release/architecture tuples.
Comment 1 Thomas Zander freebsd_committer freebsd_triage 2016-12-31 16:23:27 UTC
Valid point, especially given how much time has passed since ports r69328 . Let me test this a bit, okay?
Comment 2 commit-hook freebsd_committer freebsd_triage 2017-01-01 18:47:58 UTC
A commit references this bug:

Author: riggs
Date: Sun Jan  1 18:47:01 UTC 2017
New revision: 430269
URL: https://svnweb.freebsd.org/changeset/ports/430269

Log:
  Modernise handling of OPTIMIZED_CFLAGS

  Log:
  - Use vendor-selected OPTIMIZED_CFLAGS
  - Add them to OPTIONS_DEFAULT
  - Bump PORTREVISION

  PR:		215402
  Submitted by:	jbeich

Changes:
  head/multimedia/ffmpeg/Makefile
Comment 3 commit-hook freebsd_committer freebsd_triage 2017-01-01 18:49:01 UTC
A commit references this bug:

Author: riggs
Date: Sun Jan  1 18:48:11 UTC 2017
New revision: 430270
URL: https://svnweb.freebsd.org/changeset/ports/430270

Log:
  Reflect changes to OPTIMIZED_CFLAGS handling in multimedia/ffmpeg

  PR:		215402
  Submitted by:	jbeich

Changes:
  head/UPDATING
Comment 4 Thomas Zander freebsd_committer freebsd_triage 2017-01-01 18:57:40 UTC
Thumbs up!