Skip to content

Commit cb8bca9

Browse files
committed
try to fix clang compilation errors, 3rd attempt
1 parent 5c978b9 commit cb8bca9

File tree

5 files changed

+8
-22
lines changed

5 files changed

+8
-22
lines changed

ggml/src/ggml-cpu/arch/arm/repack.cpp

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,15 @@
11
#define GGML_COMMON_IMPL_CPP
22
#define GGML_COMMON_DECL_CPP
33
#include "ggml-common.h"
4-
#include "ggml-backend-impl.h"
5-
6-
#include "ggml-quants.h"
74
#include "ggml-impl.h"
8-
#include "ggml-cpu.h"
9-
#include "ggml-cpu-impl.h"
10-
#include "traits.h"
115

126
#include <cmath>
137
#include <cstring>
148
#include <cassert>
15-
#include <cfloat>
169
#include <cstdlib> // for qsort
1710
#include <cstdio> // for GGML_ASSERT
1811

12+
#define GGML_CPU_CLANG_WORKAROUND
1913
#include "../../repack.h"
2014

2115
#if defined(__GNUC__)
@@ -2175,4 +2169,3 @@ void ggml_gemm_iq4_nl_4x4_q8_0(int n, float * GGML_RESTRICT s, size_t bs, const
21752169
}
21762170
}
21772171
}
2178-

ggml/src/ggml-cpu/arch/riscv/repack.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
#include <cstdlib> // for qsort
1717
#include <cstdio> // for GGML_ASSERT
1818

19+
#define GGML_CPU_CLANG_WORKAROUND
1920
#include "../../repack.h"
2021

2122
#if defined(__GNUC__)
@@ -395,4 +396,3 @@ void ggml_gemm_q4_0_8x8_q8_0(int n, float * GGML_RESTRICT s, size_t bs, const vo
395396
}
396397
}
397398
}
398-

ggml/src/ggml-cpu/arch/x86/quants.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -735,7 +735,7 @@ void ggml_vec_dot_q4_1_q8_1(int n, float * GGML_RESTRICT s, size_t bs, const voi
735735
}
736736

737737
sumf = hsum_float_8(acc) + summs;
738-
738+
739739
#endif
740740
for (; ib < nb; ++ib) {
741741
int sumi0 = 0;
@@ -828,7 +828,7 @@ void ggml_vec_dot_q5_0_q8_0(int n, float * GGML_RESTRICT s, size_t bs, const voi
828828
}
829829

830830
sumf = hsum_float_8(acc);
831-
831+
832832
#endif
833833
for (; ib < nb; ++ib) {
834834
uint32_t qh;
@@ -933,7 +933,7 @@ void ggml_vec_dot_q5_1_q8_1(int n, float * GGML_RESTRICT s, size_t bs, const voi
933933
}
934934

935935
sumf = hsum_float_8(acc) + summs;
936-
936+
937937
#endif
938938
for (; ib < nb; ++ib) {
939939
uint32_t qh;

ggml/src/ggml-cpu/arch/x86/repack.cpp

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,14 @@
11
#define GGML_COMMON_IMPL_CPP
22
#define GGML_COMMON_DECL_CPP
33
#include "ggml-common.h"
4-
#include "ggml-backend-impl.h"
5-
6-
#include "ggml-quants.h"
74
#include "ggml-impl.h"
8-
#include "ggml-cpu.h"
9-
#include "ggml-cpu-impl.h"
10-
#include "traits.h"
115

12-
#include <cmath>
136
#include <cstring>
147
#include <cassert>
15-
#include <cfloat>
168
#include <cstdlib> // for qsort
179
#include <cstdio> // for GGML_ASSERT
1810

11+
#define GGML_CPU_CLANG_WORKAROUND
1912
#include "../../repack.h"
2013

2114
#if defined(__GNUC__)
@@ -3282,4 +3275,4 @@ void ggml_gemm_q4_K_8x8_q8_K(int n, float * GGML_RESTRICT s, size_t bs, const vo
32823275
}
32833276
}
32843277
#endif
3285-
}
3278+
}

ggml/src/ggml-cpu/repack.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ extern "C" {
6969
// clang++ complains: ``error: call to 'ggml_gemm_q4_0_4x4_q8_0' is ambiguous''
7070
// repro: https://godbolt.org/z/oKdeWKonM (ICE), https://godbolt.org/z/1szq6P36v (ambiguous call)
7171
// but gcc needs forward declarations
72-
#if !defined(__clang__)
72+
#if !defined(__clang__) || (defined(__clang__) && defined(GGML_CPU_CLANG_WORKAROUND))
7373
void ggml_quantize_mat_q8_0_4x4(const float * GGML_RESTRICT x, void * GGML_RESTRICT vy, int64_t k);
7474
void ggml_quantize_mat_q8_0_4x8(const float * GGML_RESTRICT x, void * GGML_RESTRICT vy, int64_t k);
7575
void ggml_quantize_mat_q8_K_4x8(const float * GGML_RESTRICT x, void * GGML_RESTRICT vy, int64_t k);

0 commit comments

Comments
 (0)