From: Louis Dionne <ldionne@apple.com>
Date: Mon, 15 Jun 2020 16:19:32 +0000 (-0400)
Subject: [libc++] Raise an exception if a Lit feature resolves to an invalid name
X-Git-Tag: llvmorg-12-init~3078
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=58610eb36808d235fa0b6fcb75a07af82d3414a0;p=platform%2Fupstream%2Fllvm.git

[libc++] Raise an exception if a Lit feature resolves to an invalid name

This allows reporting issues early when creating feature names.
---

diff --git a/libcxx/test/libcxx/selftest/dsl/dsl.sh.py b/libcxx/test/libcxx/selftest/dsl/dsl.sh.py
index 5a8ae88..b9437d6 100644
--- a/libcxx/test/libcxx/selftest/dsl/dsl.sh.py
+++ b/libcxx/test/libcxx/selftest/dsl/dsl.sh.py
@@ -194,6 +194,16 @@ class TestFeature(SetupConfigs):
         feature.enableIn(self.config)
         self.assertIn('name', self.config.available_features)
 
+    def test_name_is_not_a_string_1(self):
+        feature = dsl.Feature(name=None)
+        assert feature.isSupported(self.config)
+        self.assertRaises(ValueError, lambda: feature.enableIn(self.config))
+
+    def test_name_is_not_a_string_2(self):
+        feature = dsl.Feature(name=lambda cfg: None)
+        assert feature.isSupported(self.config)
+        self.assertRaises(ValueError, lambda: feature.enableIn(self.config))
+
     def test_adding_compile_flag(self):
         feature = dsl.Feature(name='name', compileFlag='-foo')
         origLinkFlags = copy.deepcopy(self.getSubstitution('%{link_flags}'))
diff --git a/libcxx/utils/libcxx/test/dsl.py b/libcxx/utils/libcxx/test/dsl.py
index bbed2bba..3e2d0ed 100644
--- a/libcxx/utils/libcxx/test/dsl.py
+++ b/libcxx/utils/libcxx/test/dsl.py
@@ -220,6 +220,8 @@ class Feature(object):
       config.substitutions = addTo(config.substitutions, '%{link_flags}', linkFlag)
 
     name = self._name(config) if callable(self._name) else self._name
+    if not isinstance(name, str):
+      raise ValueError("Feature did not resolve to a name that's a string, got {}".format(name))
     config.available_features.add(name)