Fix FTBFS on powerpc with recent binutils
authorSébastien Villemot <sebastien@debian.org>
Mon, 20 Dec 2021 13:15:18 +0000 (13:15 +0000)
committerSébastien Villemot <sebastien@debian.org>
Mon, 20 Dec 2021 13:15:18 +0000 (13:15 +0000)
commit718194be0d9c717c841e6d7ed2d7120ec416d308
treec55b12c126b7c3be3bccd7ef911e70761d409a67
parent5c446f0b02e3a008e78ba7baa64b894d78772e28
Fix FTBFS on powerpc with recent binutils

Forwarded: no
Last-Update: 2015-11-23

A recent modification in binutils (see
https://sourceware.org/ml/binutils/2015-04/msg00332.html) introduced a change
in the way the ppc assembly instructions "dcbt" and "dcbtst" are handled.
When compiling on a generic ppc processor, the 3-arguments form of those
instructions is no longer accepted, and one must instead use a 2-arguments
form (ignoring the TH argument).

Incidentally, the binutils change also fixed a bug in the way those
instructions were handled. On a generic ppc processor, they used to be
interpreted in the so-called "embedded" form ("dcbt TH, RA, RB", only used on
some embedded machines), while they should have been interpreted in the
so-called "server" form ("dcbt RA, RB, TH"). The ATLAS assembly directive were
apparently written in "server" form, and were therefore mis-assemblied.

Ideally, this patch should be improved in order to use the TH argument on
machines that support it (only needed in atlas_prefetch.h; other instances use
the default zero value for TH).
Last-Update: 2015-11-23
Gbp-Pq: Name powerpc-dcbt.patch
include/atlas_prefetch.h
tune/blas/gemm/CASES/ATL_cmm4x4x128_av.c
tune/blas/gemm/CASES/ATL_dmm4x4x32_ppc.c
tune/blas/gemm/CASES/ATL_dmm4x4x80_ppc.c
tune/blas/gemm/CASES/ATL_dmm8x4x2_vsx.c
tune/blas/gemm/CASES/ATL_smm4x4x128_av.c