[libcxx] Set -D_LIBCPP_HAS_NO_THREADS and -D_LIBCPP_HAS_NO_MONOTONIC_CLOCK based on available_features

[danalbert]

How are those features getting set?

[jroelofs]

I had been setting them in the lit.site.cfg, but it will work out better in zorg for them to come in from the command line too...

The updated patch lets you specify --param=additional_features=foo,bar,baz

[danalbert]

lit.cfg

296

Probably f.strip() to be safe. I don't know what lit would do with --param=additional_features=' foo, bar'

[jroelofs]

Looks like they do end up with weird whitespace in that case... So strip them on the way in.

[danalbert]

lit.cfg

361

One nitpick: libcpp-has-no-monotonic-clock to match the other one?

[jroelofs]

re-name the features for consistency

[jroelofs]

r217261

lit.cfg

                 "inferred use_clang_verify as: %r" % self.use_clang_verify)
 
     def configure_features(self):
+        additional_features = self.get_lit_conf('additional_features').split(",")
+        for f in additional_features:
+            self.config.available_features.add(f.strip())
+
         # Figure out which of the required locales we support
         locales = {
             'Darwin': {

             self.config.available_features.add(
                 'with_system_lib=%s' % sanitized_triple)
 
+        if 'libcpp-has-no-threads' in self.config.available_features:
+            self.compile_flags += ['-D_LIBCPP_HAS_NO_THREADS']
+
+        if 'no-monotonic-clock' in self.config.available_features:
+            self.compile_flags += ['-D_LIBCPP_HAS_NO_MONOTONIC_CLOCK']
+
     def configure_compile_flags(self):
         # Configure extra compiler flags.
         self.compile_flags += ['-I' + self.src_root + '/include',