Skip to content

Commit

Permalink
Revert "Ensure gcc is newer >= 11.2.1 < 12.0.0 when building knn lib (o…
Browse files Browse the repository at this point in the history
…pensearch-project#1664)"

This reverts commit da4cd1c.

Signed-off-by: Peter Zhu <[email protected]>
  • Loading branch information
peterzhuamazon committed Apr 30, 2024
1 parent a9ebd14 commit 855ff86
Showing 1 changed file with 2 additions and 10 deletions.
12 changes: 2 additions & 10 deletions scripts/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -103,26 +103,18 @@ if [ "$JAVA_HOME" = "" ]; then
echo "SET JAVA_HOME=$JAVA_HOME"
fi

# Ensure gcc version is > 11.2.1 and < 12.0.0 for faiss 1.7.4+ / SIMD Neon support on ARM64 compilation
# Ensure gcc version is at least 12.0.0 for faiss 1.7.4+ / SIMD Neon support on ARM64 compilation
# https://github.com/opensearch-project/k-NN/issues/975
# https://github.com/opensearch-project/k-NN/issues/1138
# https://github.com/opensearch-project/opensearch-build/issues/4386
# https://github.com/opensearch-project/opensearch-build/issues/4379#issuecomment-2067191682
# https://github.com/opensearch-project/opensearch-build/issues/4379#issuecomment-2083623882
# https://github.com/opensearch-project/opensearch-build/issues/4379#issuecomment-2084133839
GCC_VERSION=`gcc --version | head -n 1 | cut -d ' ' -f3`
GCC_REQUIRED_VERSION=11.2.1
GCC_REQUIRED_VERSION=12.0.0
COMPARE_VERSION=`echo $GCC_REQUIRED_VERSION $GCC_VERSION | tr ' ' '\n' | sort -V | uniq | head -n 1`
if [ "$COMPARE_VERSION" != "$GCC_REQUIRED_VERSION" ]; then
echo "gcc version on this env is older than $GCC_REQUIRED_VERSION, exit 1"
exit 1
fi
GCC_REQUIRED_VERSION_CEILING=12.0.0
COMPARE_VERSION_CEILING=`echo $GCC_REQUIRED_VERSION_CEILING $GCC_VERSION | tr ' ' '\n' | sort -V | uniq | tail -n 1`
if [ "$COMPARE_VERSION_CEILING" != "$GCC_REQUIRED_VERSION_CEILING" ] && (echo "$OSTYPE" | grep -i linux); then
echo "gcc version on this env is newer than $GCC_REQUIRED_VERSION_CEILING, exit 1"
exit 1
fi

# Build k-NN lib and plugin through gradle tasks
cd $work_dir
Expand Down

0 comments on commit 855ff86

Please sign in to comment.