Bug 238868 - math/openblas: fix build on powerpc64 with TARGET=PPC970 and switch to PPC970
Summary: math/openblas: fix build on powerpc64 with TARGET=PPC970 and switch to PPC970
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: Piotr Kubaj
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-06-28 15:38 UTC by Piotr Kubaj
Modified: 2019-07-30 12:11 UTC (History)
1 user (show)

See Also:
phd_kimberlite: maintainer-feedback+

Attachments
patch (2.50 KB, patch)
2019-06-28 15:38 UTC, Piotr Kubaj 		pkubaj: maintainer-approval? (phd_kimberlite)
 Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Piotr Kubaj freebsd_committer freebsd_triage 2019-06-28 15:38:44 UTC 
Created attachment 205402 [details]
patch

This patch ( https://github.com/xianyi/OpenBLAS/pull/2170 ) fixes build for PPC970 target. Since most people that run FreeBSD on powerpc64 use PowerPC 970 CPU's, switch to it by default and add non-default option to optimize for POWER6.

Technically, it would require bumping PORTREVISION, but considering low popularity of powerpc64 platform, I didn't do it. Do you require it?
Comment 1 commit-hook freebsd_committer freebsd_triage 2019-07-30 12:11:38 UTC 
A commit references this bug:

Author: pkubaj
Date: Tue Jul 30 12:11:22 UTC 2019
New revision: 507634
URL: https://svnweb.freebsd.org/changeset/ports/507634

Log:
  math/openblas: fix build on powerpc64 with TARGET=PPC970 and switch to PPC970

  This patch fixes build for PPC970 target. Since most people that run FreeBSD on powerpc64 use PowerPC 970 CPU's, switch to it by default and add non-default option to optimize for POWER6 (optimizing for newer POWER doesn't work on BE).

  Bump PORTREVISION because of package change.

  PR:		238868
  Approved by:	phd_kimberlite@yahoo.co.jp (maintainer), linimon (mentor)
  Differential Revision:	https://reviews.freebsd.org/D21010

Changes:
  head/math/openblas/Makefile
  head/math/openblas/files/patch-common__power.h
  head/math/openblas/files/patch-param.h