Reduce unnecessary checks

If `have_header` returns `false`, `have_func` using that header also return `false` always, because that header cannot be included.
ruby · Sep 9, 2024 · 8fa7712 · 8fa7712
1 parent 33b5374
commit 8fa7712
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/ext/bigdecimal/extconf.rb b/ext/bigdecimal/extconf.rb
@@ -24,15 +24,17 @@ def have_builtin_func(name, check_expr, opt = "", &b)
 have_header("stdbool.h")
 have_header("stdlib.h")
 
-have_header("x86intrin.h")
-have_func("_lzcnt_u32", "x86intrin.h")
-have_func("_lzcnt_u64", "x86intrin.h")
-
-have_header("intrin.h")
-have_func("__lzcnt", "intrin.h")
-have_func("__lzcnt64", "intrin.h")
-have_func("_BitScanReverse", "intrin.h")
-have_func("_BitScanReverse64", "intrin.h")
+if have_header("x86intrin.h")
+  have_func("_lzcnt_u32", "x86intrin.h")
+  have_func("_lzcnt_u64", "x86intrin.h")
+end
+
+if have_header("intrin.h")
+  have_func("__lzcnt", "intrin.h")
+  have_func("__lzcnt64", "intrin.h")
+  have_func("_BitScanReverse", "intrin.h")
+  have_func("_BitScanReverse64", "intrin.h")
+end
 
 have_func("labs", "stdlib.h")
 have_func("llabs", "stdlib.h")