depenencies/llvm: Handle llvm-config --shared-mode failing (#7379)

* depenencies/llvm: Handle llvm-config --shared-mode failing

Fixes: #7371
Fixes: #7878

* test cases/llvm: Refactor to use test.json

Instead of trying to cover everything internally

3 files changed, 55 insertions, 39 deletions

diff --git a/test cases/frameworks/15 llvm/meson.build b/test cases/frameworks/15 llvm/meson.build
index 4b2c88c..bae83c5 100644
--- a/test cases/frameworks/15 llvm/meson.build
+++ b/test cases/frameworks/15 llvm/meson.build
@@ -1,28 +1,23 @@
 project('llvmtest', ['c', 'cpp'], default_options : ['c_std=c99'])
 
-d = dependency('llvm', required : false, method : 'config-tool')
+method = get_option('method')
+static = get_option('link-static')
+d = dependency('llvm', required : false, method : method, static : static)
 if not d.found()
-  d = dependency('llvm', required : false, static : true)
-  if not d.found()
-    error('MESON_SKIP_TEST llvm not found.')
-  else
-    static = true
-  endif
-else
-  static = false
+  error('MESON_SKIP_TEST llvm not found.')
 endif
 
-d = dependency('llvm', modules : 'not-found', required : false, static : static)
+d = dependency('llvm', modules : 'not-found', required : false, static : static, method : method)
 assert(d.found() == false, 'not-found llvm module found')
 
-d = dependency('llvm', version : '<0.1', required : false, static : static)
+d = dependency('llvm', version : '<0.1', required : false, static : static, method : method)
 assert(d.found() == false, 'ancient llvm module found')
 
-d = dependency('llvm', optional_modules : 'not-found', required : false, static : static)
+d = dependency('llvm', optional_modules : 'not-found', required : false, static : static, method : method)
 assert(d.found() == true, 'optional module stopped llvm from being found.')
 
 # Check we can apply a version constraint
-d = dependency('llvm', version : ['< 500', '>=@0@'.format(d.version())], required: false, static : static)
+d = dependency('llvm', version : ['< 500', '>=@0@'.format(d.version())], required: false, static : static, method : method)
 assert(d.found() == true, 'Cannot set version constraints')
 
 dep_tinfo = dependency('tinfo', required : false)
@@ -31,29 +26,23 @@ if not dep_tinfo.found()
   dep_tinfo = cpp.find_library('tinfo', required: false)
 endif
 
-foreach method : ['config-tool', 'cmake']
-  foreach static : [true, false]
-    message('Trying method @0@ for @1@ link'.format(method, static ? 'static' : 'dynamic'))
-    llvm_dep = dependency(
-      'llvm',
-      modules : ['bitwriter', 'asmprinter', 'executionengine', 'target',
-                'mcjit', 'nativecodegen', 'amdgpu'],
-      required : false,
-      static : static,
-      method : method,
-    )
-    if llvm_dep.found()
-      name = static ? 'static' : 'dynamic'
-      executable(
-        'sum-@0@-@1@'.format(name, method),
-        'sum.c',
-        dependencies : [
-          llvm_dep, dep_tinfo,
-          # zlib will be statically linked on windows
-          dependency('zlib', required : host_machine.system() != 'windows'),
-          meson.get_compiler('c').find_library('dl', required : false),
-        ]
-      )
-    endif
-  endforeach
-endforeach
+llvm_dep = dependency(
+  'llvm',
+  modules : ['bitwriter', 'asmprinter', 'executionengine', 'target',
+             'mcjit', 'nativecodegen', 'amdgpu'],
+  required : false,
+  static : static,
+  method : method,
+)
+if llvm_dep.found()
+  executable(
+    'sum',
+    'sum.c',
+    dependencies : [
+      llvm_dep, dep_tinfo,
+      # zlib will be statically linked on windows
+      dependency('zlib', required : host_machine.system() != 'windows'),
+      meson.get_compiler('c').find_library('dl', required : false),
+    ]
+  )
+endif
diff --git a/test cases/frameworks/15 llvm/meson_options.txt b/test cases/frameworks/15 llvm/meson_options.txt
new file mode 100644
index 0000000..de3d172
--- /dev/null
+++ b/test cases/frameworks/15 llvm/meson_options.txt
@@ -0,0 +1,10 @@
+option(
+  'method',
+  type : 'combo',
+  choices : ['config-tool', 'cmake']
+)
+option(
+  'link-static',
+  type : 'boolean',
+  value : false,
+)
diff --git a/test cases/frameworks/15 llvm/test.json b/test cases/frameworks/15 llvm/test.json
new file mode 100644
index 0000000..0cad05a
--- /dev/null
+++ b/test cases/frameworks/15 llvm/test.json
@@ -0,0 +1,17 @@
+{
+  "matrix": {
+    "options": {
+      "method": [
+        { "val": "config-tool" },
+        { "val": "cmake" }
+      ],
+      "link-static": [
+        { "val": true },
+        { "val": false }
+      ]
+    },
+    "exclude": [
+      { "method": "cmake", "link-static": false }
+    ]
+  }
+}