[libc++] Reduce the verbosity when running the libc++ Lit configuration
authorLouis Dionne <ldionne.2@gmail.com>
Mon, 30 May 2022 15:06:19 +0000 (11:06 -0400)
committerLouis Dionne <ldionne.2@gmail.com>
Mon, 30 May 2022 16:03:24 +0000 (12:03 -0400)
We print the same information as before, however we do it with less
verbosity unless `--debug` is used.

libcxx/utils/libcxx/test/newconfig.py

index 6444387..2ac5cf2 100644 (file)
@@ -13,6 +13,8 @@ def _getSubstitution(substitution, config):
   raise ValueError('Substitution {} is not in the config.'.format(substitution))
 
 def configure(parameters, features, config, lit_config):
+  note = lambda s: lit_config.note("({}) {}".format(config.name, s))
+
   # Apply the actions supplied by parameters to the configuration first, since
   # parameters are things that we request explicitly and which might influence
   # what features are implicitly made available next.
@@ -20,19 +22,24 @@ def configure(parameters, features, config, lit_config):
     actions = param.getActions(config, lit_config.params)
     for action in actions:
       action.applyTo(config)
-      lit_config.note("Applied '{}' as a result of parameter '{}'".format(
-        action.pretty(config, lit_config.params),
-        param.pretty(config, lit_config.params)))
+      if lit_config.debug:
+        note("Applied '{}' as a result of parameter '{}'".format(
+          action.pretty(config, lit_config.params),
+          param.pretty(config, lit_config.params)))
 
   # Then, apply the automatically-detected features.
   for feature in features:
     actions = feature.getActions(config)
     for action in actions:
       action.applyTo(config)
-      lit_config.note("Applied '{}' as a result of implicitly detected feature '{}'".format(
-        action.pretty(config, lit_config.params),
-        feature.pretty(config)))
+      if lit_config.debug:
+        note("Applied '{}' as a result of implicitly detected feature '{}'".format(
+          action.pretty(config, lit_config.params),
+          feature.pretty(config)))
 
   # Print the basic substitutions
   for sub in ('%{cxx}', '%{flags}', '%{compile_flags}', '%{link_flags}', '%{exec}'):
-    lit_config.note("Using {} substitution: '{}'".format(sub, _getSubstitution(sub, config)))
+    note("Using {} substitution: '{}'".format(sub, _getSubstitution(sub, config)))
+
+  # Print all available features
+  note("All available features: {}".format(', '.join(config.available_features)))