[libcxx-commits] [PATCH] D111202: [libc++] Add from-scratch testing configs for Windows

[Louis Dionne via Phabricator via libcxx-commits]

ldionne requested changes to this revision.
ldionne added inline comments.
This revision now requires changes to proceed.


================
Comment at: libcxx/test/configs/llvm-libc++-mingw.cfg.in:14
+config.substitutions.append(('%{link_flags}',
+    '%{user_link_flags} -nostdlib++ -L %{lib} -lc++'
+))
----------------
There's a weird character here.


================
Comment at: libcxx/utils/libcxx/test/format.py:24
     """
-    command = "%{{cxx}} -xc++ {} -Werror -fsyntax-only -Xclang -verify-ignore-unexpected".format(os.devnull)
+    command = "%{{cxx}} %{{flags}} -xc++ {} -Werror -fsyntax-only -Xclang -verify-ignore-unexpected".format(os.devnull)
     command = lit.TestRunner.applySubstitutions([command], config.substitutions,
----------------
If we do this, I think it should then be `%{{flags}} %{{compile_flags}}`.


================
Comment at: libcxx/utils/libcxx/test/newconfig.py:14-17
+def quote(s):
+    if platform.system() == 'Windows':
+        return lit.TestRunner.quote_windows_command([s])
+    return pipes.quote(s)
----------------
I suggest we simply move that to `lit.TestRunner`. It's too general purpose to be included in `newconfig.py`, even though I know that is convenient.


================
Comment at: libcxx/utils/libcxx/test/newconfig.py:50
+
+  sys.stderr.flush()  # Force flushing to avoid broken output on Windows
----------------
If this is necessary, IMO this should be part of `lit_config.note()` itself.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111202/new/

https://reviews.llvm.org/D111202