Merge pull request #1944 from hartzell/patch-1

Typo: Skyalke -> Skylake

Typo: Skyalke -> Skylake

Worth fixing, it gets in the way of searching....

Merge pull request #1939 from TiborGY/patch-2

Fix typo in UNKNOWN core name

Merge pull request #1943 from martin-frbg/issue1748

Re-enable loop unrolling in trmv and remove the scary warning

Re-enable loop unrolling in trmv and remove the scary warning

fixes #1748 as that half of the fix for #1332 appears to have been an overreaction on my part.

Merge pull request #1942 from martin-frbg/issue1720

Delete the pthread key on cleanup in TLS mode

Remove stray include of complex.h

already provided conditionally by common.h via openblas_utest.h
Unconditional inclusion breaks older Android and similar platforms that use OPENBLAS_COMPLEX_STRUCT

Delete the pthread key on cleanup in TLS mode

to avoid a crash when OpenBLAS was loaded via dlopen and libc tries to clean up the leaked TLS after dlclose
Fixes #1720

Fix wrong case in TARGET setting for Alpine

Update cpuid_mips64.c

Update Makefile

Update cpuid_mips.c

Update cpuid_arm.c

Fix typo in UNKNOWN core name

Should be of no consequence, right?

Merge pull request #1932 from martin-frbg/issue1915

Add -fPIC to provided CFLAGS/FFLAGS if required

Remove unnecessary change again

Merge pull request #1934 from fenrus75/betagoof

Fix thinko in skylake beta handling

Fix thinko in skylake beta handling

casting ints is cheaper but it has a rounding, not memory casing effect, resulting in
invalid outcome

Make sure that -fPIC is present if needed

Make sure that -fPIC is present when needed

override user-provided FFLAGS if necessary

Merge pull request #1931 from martin-frbg/pr1921

Add -mavx2 to TARGET=HASWELL builds

Use -dumpversion with gcc only

Add -mavx2 for TARGET=HASWELL if compiler supports and requires it

Add -mavx2 to TARGET=HASWELL builds

to leverage improvements from PR#1921

Merge pull request #1930 from martin-frbg/issue1908

Reflect ARMV8 target definition changes from PR1876

Reflect ARMV8 target definition changes from PR1876

and create config target directory for cross-compiles.

Merge pull request #1929 from martin-frbg/issue1924

Avoid taking the root of a negative number in simple threaded syrk

Avoid taking the root of a negative number

Fixes #1924 where numpy 1.17+ would report the (transient) FE_INVALID exception raised for the domain error.

Merge pull request #1 from xianyi/develop

rebase

Merge pull request #1921 from fenrus75/haswelldgemm

Replicate some of the SKYLAKEX dgemm improvements also to HASWELL

set GEMM_PREFERED_SIZE for HASWELL

Haswell likes a GEMM_PREFERED_SIZE of 16 to improve the split that the
threading code does to make it a nice multiple of the SIMD kernel size

dgemm: use dgemm_ncopy_8_skylakex.c also for Haswell

The dgemm_ncopy_8_skylakex.c code is not avx512 specific and gives
a nice performance boost for medium sized matrices

dgemm: Use the skylakex beta function also for haswell

it's more efficient for certain tall/skinny matrices

Merge pull request #1919 from fenrus75/haswelltuning

(sgemm) Apply some of the SKYLAKEX optimizations also to HASWELL

Update Makefile

typo

Add -mavx2 on Haswell only if the compiler supports it

Make the skylakex/haswell sgemm code compile and run even with compilers without avx2 support

use -maxv2 on haswell

should be core-avx2

fix elseifeq and use older option core2-avx for compatibility

Add -march=haswell to HASWELL part of DYNAMIC_ARCH build

Use sgemm_ncopy_4_skylakex.c also for Haswell

sgemm_ncopy_4_skylakex.c uses SSE transpose operations where the
real perf win happens; this also works great for Haswell.

This gives double digit percentage gains on small and skinny matrices

Use the skylake sgemm beta code also for haswell

with a few small changes it's possible to use the skylake sgemm code
also for haswell, this gives a modest gain (10% range) for smallish
matrixes but does wonders for very skinny matrixes

Merge pull request #1914 from fenrus75/smallmatrix

Add a "sgemm direct" mode for small matrixes

Add a "sgemm direct" mode for small matrixes

OpenBLAS has a fancy algorithm for copying the input data while laying
it out in a more CPU friendly memory layout.

This is great for large matrixes; the cost of the copy is easily
ammortized by the gains from the better memory layout.

But for small matrixes (on CPUs that can do efficient unaligned loads) this
copy can be a net loss.

This patch adds (for SKYLAKEX initially) a "sgemm direct" mode, that bypasses
the whole copy machinary for ALPHA=1/BETA=0/... standard arguments,
for small matrixes only.

What is small? For the non-threaded case this has been measured to be
in the M*N*K = 28 * 512 * 512 range, while in the threaded case it's
less, around M*N*K = 1 * 512 * 512

Merge pull request #1910 from martin-frbg/issue1909

Fix for DYNAMIC_ARCH builds made on a AVX512-capable host

make sure the added march=skylake-avx512 does not cause problems on Windows

Add -march=skylake-avx512 to SkylakeX part of DYNAMIC_ARCH builds

Avoid adding blanket march=skylake-avx512 to dynamic_arch builds

Fix typo in previous commit for arm dynamic arch

Add DYNAMIC_CORE list for ARM64

cf #1908

Make sure to use the arm version of dynamic.c in ARM64 DYNAMIC_ARCH

cf. #1908

Merge pull request #1904 from martin-frbg/issue1870

Fix cmake parsing of GEMM kernels for ARMV8

Fix missing parameter in popen call

Rewrite the conditional for OSX to fix cmake parsing on others

The Makefile variable parser in utils.cmake currently does not handle conditionals. Having the definitions for non-OSX last will at least make cmake builds work again on non-OSX platforms.

Avoid creating spurious non-suffixed c/zgemm_kernels

Plain cgemm_kernel and zgemm_kernel are not used anywhere, only cgemm_kernel_b etc.
Needlessly building them (without any define like NN, CN, etc.) just happened to work on most platforms, but not on arm64. See #1870

Fix  typo that broke CNRM2 on ARMV8 since 0.3.0

must have happened in my #1449

Merge pull request #1903 from rengolin/armv8

Fix two mistakes on Arm64 builds

Fix two mistakes on Arm64 builds

 * Falkor is an ARMv8.0 with ARMv8.1 features, and chosing armv8.1-a for
   march generates instructions it cannot cope with. Reverting it back
   to armv8-a.
 * ThunderX2's build was left with a #define VULCAN, which made it miss
   the right compiler flags in Makefile.arm64, although it did create
   the right library in the end.

Use p2align instead of align for OSX compatibility

fixes #1902

Merge pull request #1899 from brada4/fbsd12

Add mutually supported architecture mappings for FreeBSD12 ports

Merge branch 'develop' into fbsd12

Merge pull request #1894 from pkubaj/patch-2

Use correct ARCH name on BSD powerpc64

Update Makefile.system

Update with the changes from 0.3.4

Increment version to 0.3.5.dev

Increment version to 0.3.5.dev

Merge branch 'release-0.3.0' into develop

fixup BSD naming of powerpc arch

oops

Add architecture mappings for FreeBSD12

init

Handle Android environments that identify as Linux (#1898)

* Handle Android environments that identify as Linux

termux terminal emulator does this, causing build failures through missed defines in common.h

Really revert xDOT changes from 1832

neglected to rebase #1892 on merging

Merge pull request #1892 from martin-frbg/mipsdot

revert MIPS64 xDOT kernel changes from #1832

Fix build on POWER, remove DragonFly, add NetBSD

__asm is complete on its own

DBSD developers state they will only support amd64, but NetBSD supports POWER.

Use correct Makefile on powerpc64

FreeBSD uses powerpc64 name for POWER architecture. Use correct Makefile for this platform.

remove surplus locking code , only enabled w x86, disabled or never enabled on all others

Revert DOT kernel changes from #1834

as the failures seen on Loongson3A appear to be limited to DSDOT/SDSDOT (i.e. my hackish "fix" from #1684)

Use generic kernel for DSDOT/SDSDOT

as discussed in #1834

Merge pull request #1834 from fengrl/develop

register push/pop command change

Merge pull request #1890 from martin-frbg/issue1889

Include version number in openblas_get_config output

call it OpenBLAS not just version

Improve formatting of config output

Propagate version number for openblas_get_config

propagate verison number for openblas_config_version

Add version information to openblas_get_config output

Merge pull request #1885 from brada4/freebsd

Fix freebsd clang compilation of skylakex

Add return value so that freebsd system clang does not err out

init

Merge pull request #1875 from martin-frbg/issue1851

Serialize accesses to parallelized level3 functions from multiple cal…

Merge pull request #1869 from martin-frbg/axpy0

Handle special case INCX=0,INCY=0 in the axpy interface

Merge pull request #1878 from kiwifb/PGI_f_check

Correct link flags for PGI compiler.

Merge pull request #1876 from rengolin/armv8-cleanup

Simplifying ARMv8 build parameters

Correct link flags for PGI compiler.

Merge pull request #1868 from martin-frbg/aix_cpuid

Use prtconf to determine CPU type on AIX

fix missing parenthesis

Add CriticalSection handling instead of mutexes for Windows

Simplifying ARMv8 build parameters

ARMv8 builds were a bit mixed up, with ThunderX2 code in ARMv8 mode
(which is not right because TX2 is ARMv8.1) as well as requiring a few
redundancies in the defines, making it harder to maintain and understand
what core has what. A few other minor issues were also fixed.

Tests were made on the following cores: A53, A57, A72, Falkor, ThunderX,
ThunderX2, and XGene.

Tests were: OpenBLAS/test, OpenBLAS/benchmark, BLAS-Tester.

A summary:
 * Removed TX2 code from ARMv8 build, to make sure it is compatible with
   all ARMv8 cores, not just v8.1. Also, the TX2 code has actually
   harmed performance on big cores.
 * Commoned up ARMv8 architectures' defines in params.h, to make sure
   that all will benefit from ARMv8 settings, in addition to their own.
 * Adding a few more cores, using ARMv8's include strategy, to benefit
   from compiler optimisations using mtune. Also updated cache
   information from the manuals, making sure we set good conservative
   values by default. Removed Vulcan, as it's an alias to TX2.
 * Auto-detecting most of those cores, but also updating the forced
   compilation in getarch.c, to make sure the parameters are the same
   whether compiled natively or forced arch.

Benefits:
 * ARMv8 build is now guaranteed to work on all ARMv8 cores
 * Improved performance for ARMv8 builds on some cores (A72, Falkor,
   ThunderX1 and 2: up to 11%) over current develop
 * Improved performance for *all* cores comparing to develop branch
   before TX2's patch (9% ~ 36%)
 * ThunderX1 builds are 14% faster than ARMv8 on TX1, 9% faster than
   current develop's branch and 8% faster than deveop before tx2 patches

Issues:
 * Regression from current develop branch for A53 (-12%) and A57 (-3%)
   with ARMv8 builds, but still faster than before TX2's commit (+15%
   and +24% respectively). This can be improved with a simplification of
   TX2's code, to be done in future patches. At least the code is
   guaranteed to be ARMv8.0 now.

Comments:
 * CortexA57 builds are unchanged on A57 hardware from develop's branch,
   which makes sense, as it's untouched.
 * CortexA72 builds improve over A57 on A72 hardware, even if they're
   using the same includes due to new compiler tunning in the makefile.