From 64d3e3f3ba6cbc40731914fc205a5a292d145d42 Mon Sep 17 00:00:00 2001 From: Yagiz Nizipli Date: Sat, 17 Dec 2022 15:00:18 -0500 Subject: [PATCH] deps: disable avx512 for simutf on benchmark ci Co-authored-by: Daniel Lemire PR-URL: https://github.com/nodejs/node/pull/45803 Reviewed-By: Robert Nagy Reviewed-By: Matteo Collina Reviewed-By: Anna Henningsen Reviewed-By: Michael Dawson --- deps/simdutf/simdutf.gyp | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/deps/simdutf/simdutf.gyp b/deps/simdutf/simdutf.gyp index baa6ad7e437956..93c83e618c7a8f 100644 --- a/deps/simdutf/simdutf.gyp +++ b/deps/simdutf/simdutf.gyp @@ -1,4 +1,9 @@ { + 'variables': { + 'gas_version%': '0.0', + 'nasm_version%': '0.0', + 'llvm_version%': '0.0', + }, 'targets': [ { 'target_name': 'simdutf', @@ -8,6 +13,16 @@ 'include_dirs': ['.'], }, 'sources': ['simdutf.cpp'], + 'conditions': [ + ['OS=="linux"', { + 'conditions': [ + # TODO(anonrig): Remove this validation when benchmark CI has binutils >= 2.30 + ['v(gas_version) < v("2.30") and v(nasm_version) < v("2.14") and v(llvm_version) < v("6.0")', { + 'defines': ['SIMDUTF_IMPLEMENTATION_ICELAKE=0'], + }], + ], + }], + ], }, ] }