fix crashes Appel-M CPU, probably a bug from the compiler

Author: mgravey

build/Makefile

@@ -4,7 +4,7 @@ export GIT_URL=$(shell sh -c 'git config --get remote.origin.url || echo "https:
 
 export SRCS=$(BUILD_DIR)/../src
 export CFLAGS=-fPIC -O3 -std=c++11 -Wno-deprecated -Wno-unused -Wno-deprecated-declarations
-export CXXFLAGS=
+export CXXFLAGS= -Wno-vla-extension #-flto
 export INC=-I$(BUILD_DIR)/../include
 ifndef HOMEBREW_PRODUCT
 	export LIBINC=-I/usr/include -I/usr/include/jsoncpp -I/opt/local/include

include/fKb.hpp

@@ -34,7 +34,7 @@
 namespace fKst {
 
 template<typename T>
-	inline __attribute__((always_inline)) void addValueB(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int i){
+	inline /*__attribute__((always_inline))*/ void addValueB(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int i){
 		short position=k-2;
 		while ((data[i]>output[position]) && (position>-1) ){
 			output[position+1]=output[position];
@@ -44,7 +44,7 @@ template<typename T>
 	}
 
 template<typename T>
-	inline __attribute__((always_inline)) void addValueB(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int* restrict positionValue, unsigned int i){
+	inline /*__attribute__((always_inline))*/ void addValueB(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int* restrict positionValue, unsigned int i){
 		short position=k-2;
 		while ((data[i]>output[position]) && (position>-1) ){
 			output[position+1]=output[position]; 
@@ -56,7 +56,7 @@ template<typename T>
 	}
 
 template<typename T, typename urgT>
-	inline __attribute__((always_inline)) void addValueB(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int* restrict positionValue, urgT generator, unsigned &cpt, unsigned int i){
+	inline /*__attribute__((always_inline))*/ void addValueB(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int* restrict positionValue, urgT generator, unsigned &cpt, unsigned int i){
 		short position=k-2;
 		short positionLikeLast=k-2;
 		while ((output[k-1]==output[positionLikeLast]) && (position>-1) ){
@@ -841,14 +841,14 @@ inline void findKBigest(const T* data,const unsigned int N,const unsigned short
 	}
 #endif
 
-#if __arm64__
-	{
-		if(std::is_same<T, float>::value){
-			findKbigestARM(data, N, k, output);
-			return;
-		}
-	}
-#endif
+// #if __arm64__
+// 	{
+// 		if(std::is_same<T, float>::value){
+// 			findKbigestARM(data, N, k, output);
+// 			return;
+// 		}
+// 	}
+// #endif
 
 	findKbigest<T>(data, N, k, output);
 	return;
@@ -890,14 +890,14 @@ inline void findKBigest(const T* data,const unsigned int N,const unsigned short
 	}
 #endif
 
-#if __arm64__
-	{
-		if(std::is_same<T, float>::value){
-			findKbigestARM(data, N, k, output, positionValue);
-			return;
-		}
-	}
-#endif
+// #if __arm64__
+// 	{
+// 		if(std::is_same<T, float>::value){
+// 			findKbigestARM(data, N, k, output, positionValue);
+// 			return;
+// 		}
+// 	}
+// #endif
 
 	findKbigest<T>(data, N, k, output, positionValue);
 	return;
@@ -939,14 +939,14 @@ inline void findKBigest(const T* data,const unsigned int N,const unsigned short
 	}
 #endif
 
-#if __arm64__
-	{
-		if(std::is_same<T, float>::value){
-			findKbigestARM(data, N, k, output, positionValue, generator);
-			return;
-		}
-	}
-#endif
+// #if __arm64__
+// 	{
+// 		if(std::is_same<T, float>::value){
+// 			findKbigestARM(data, N, k, output, positionValue, generator);
+// 			return;
+// 		}
+// 	}
+// #endif
 
 	findKbigest<T>(data, N, k, output, positionValue, generator);
 	return;

include/fKs.hpp

@@ -35,7 +35,7 @@
 namespace fKst {
 
 template<typename T>
-inline __attribute__((always_inline)) void addValueS(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int i){
+inline /*__attribute__((always_inline))*/ void addValueS(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int i){
 	short position=k-2;
 	while ((data[i]<output[position]) && (position>-1) ){
 		output[position+1]=output[position];
@@ -45,7 +45,7 @@ inline __attribute__((always_inline)) void addValueS(const T* data,const unsigne
 }
 
 template<typename T>
-inline __attribute__((always_inline)) void addValueS(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int* restrict positionValue, unsigned int i){
+inline /*__attribute__((always_inline))*/ void addValueS(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int* restrict positionValue, unsigned int i){
 	short position=k-2;
 	while ((data[i]<output[position]) && (position>-1) ){
 		output[position+1]=output[position]; 
@@ -57,7 +57,7 @@ inline __attribute__((always_inline)) void addValueS(const T* data,const unsigne
 }
 
 template<typename T, typename urgT>
-inline __attribute__((always_inline)) void addValueS(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int* restrict positionValue, urgT generator, unsigned &cpt, unsigned int i){
+inline /*__attribute__((always_inline))*/ void addValueS(const T* data,const unsigned int N,const unsigned short k, T* restrict output, unsigned int* restrict positionValue, urgT generator, unsigned &cpt, unsigned int i){
 	short position=k-2;
 	short positionLikeLast=k-2;
 	while ((output[k-1]==output[positionLikeLast]) && (position>-1) ){