Skip to content

Commit 8f8fa56

Browse files
committed
Add simd utils
1 parent 300a8e9 commit 8f8fa56

File tree

4 files changed

+6
-7
lines changed

4 files changed

+6
-7
lines changed

.gitmodules

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,3 +23,6 @@
2323
[submodule "src/3rd_party/ruy"]
2424
path = src/3rd_party/ruy
2525
url = https://github.com/google/ruy.git
26+
[submodule "src/3rd_party/simd_utils"]
27+
path = src/3rd_party/simd_utils
28+
url = https://github.com/JishinMaster/simd_utils.git

src/3rd_party/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ add_subdirectory(./faiss)
99
include_directories(./faiss)
1010

1111
if(COMPILE_CPU)
12-
if((NOT ARM) AND (NOT GENERATE_MARIAN_INSTALL_TARGETS))
12+
if((NOT ${CMAKE_TARGET_ARCHITECTURE_CODE} MATCHES "arm") AND (NOT GENERATE_MARIAN_INSTALL_TARGETS))
1313
set(INTGEMM_DONT_BUILD_TESTS ON CACHE BOOL "Disable intgemm tests")
1414
add_subdirectory(./intgemm)
1515
endif()

src/3rd_party/simd_utils

Submodule simd_utils added at c092ef9

src/tensors/cpu/prod_blas.h

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -91,12 +91,7 @@ inline void GemmRuy(const bool transA,
9191
}
9292

9393
} else {
94-
// @jerinphilip has not yet been able to find a ruy primitive that does in
95-
// place addition to obtain full gemm.
96-
//
97-
// Safe bet is to make an additional allocation to store the result of
98-
// multiply and use the existing values in C.
99-
//
94+
// No multiply-add in Ruy
10095
// See also: https://github.com/google/ruy/issues/307
10196

10297
AlignedVector<float> intermediate(M * N);

0 commit comments

Comments
 (0)