Skip to content

Carry an openblas patch to fix bug in power kernels#21091

Merged
staticfloat merged 2 commits intomasterfrom
tk/openblas-powerfix
Mar 19, 2017
Merged

Carry an openblas patch to fix bug in power kernels#21091
staticfloat merged 2 commits intomasterfrom
tk/openblas-powerfix

Conversation

@tkelman
Copy link
Copy Markdown
Contributor

@tkelman tkelman commented Mar 19, 2017

Reverts #20165 because this was a bug in openblas, not gcc or suitesparse. X-ref #20123 (comment), I bisected on this in openblas (with a cross-compile using qemu, hopefully that shouldn't make a difference though) and identified OpenMathLib/OpenBLAS#1098 as fixing the problem. It's power-specific and applies cleanly to 0.2.19, so let's swap patches, carry the openblas power kernel fixes until there's a new release available.

tkelman added 2 commits March 19, 2017 09:05
This reverts commit 145eae8.

This was actually a bug in the power kernels in OpenBLAS,
not a GCC or SuiteSparse problem
@tkelman tkelman added backport pending 0.5 bugfix This change fixes an existing bug upstream The issue is with an upstream dependency, e.g. LLVM system:powerpc PowerPC labels Mar 19, 2017
@tkelman
Copy link
Copy Markdown
Contributor Author

tkelman commented Mar 19, 2017

Someone with access to a power system should verify that tests pass on this branch after a
make -C deps distclean-openblas distclean-suitesparse

@tkelman tkelman requested a review from staticfloat March 19, 2017 13:47
@staticfloat
Copy link
Copy Markdown
Member

@jlbuild !filter=ppc !nuke

@jlbuild
Copy link
Copy Markdown

jlbuild commented Mar 19, 2017

Status of d1cd951 builds:

Builder Name Nuke Build Download
linuxppc64le COMPLETE COMPLETE Download

@staticfloat staticfloat merged commit e6ef51f into master Mar 19, 2017
@tkelman tkelman deleted the tk/openblas-powerfix branch March 19, 2017 20:09
tkelman added a commit that referenced this pull request May 2, 2017
tkelman added a commit that referenced this pull request May 3, 2017
from OpenMathLib/OpenBLAS#1098

(cherry picked from commit 26beab3)
ref #21091

fix Makefile.system dependency for release-0.5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This change fixes an existing bug system:powerpc PowerPC upstream The issue is with an upstream dependency, e.g. LLVM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants