Use the two-operand form of DCBT on all PPC970 regardless of OS
authorMartin Kroeker <martin@ruby.chemie.uni-freiburg.de>
Sun, 3 Nov 2019 21:55:31 +0000 (22:55 +0100)
committerGitHub <noreply@github.com>
Sun, 3 Nov 2019 21:55:31 +0000 (22:55 +0100)
There seems to be no advantage to the three-operand form used in the earliest GotoBLAS kernels, and it causes compilation problems  on other than the previously special-cased platforms as well

common_power.h

index 5e15b75..bcfc209 100644 (file)
@@ -241,7 +241,7 @@ static inline int blas_quickdivide(blasint x, blasint y){
 #define HAVE_PREFETCH
 #endif
 
-#if defined(POWER3) || defined(POWER6) || defined(PPCG4) || defined(CELL) || defined(POWER8) || defined(POWER9) || ( defined(PPC970) && ( defined(OS_DARWIN) || defined(OS_FREEBSD) ) )
+#if defined(POWER3) || defined(POWER6) || defined(PPCG4) || defined(CELL) || defined(POWER8) || defined(POWER9) || defined(PPC970)
 #define DCBT_ARG       0
 #else
 #define DCBT_ARG       8