labath added inline comments.

================
Comment at: lit/Expr/TestIRMemoryMapWindows.test:1-12
+# REQUIRES: windows
+
+# RUN: clang-cl /Zi %p/Inputs/call-function.cpp -o %t
+
+# RUN: lldb-test ir-memory-map %t %S/Inputs/ir-memory-map-basic
+# RUN: lldb-test ir-memory-map -host-only %t %S/Inputs/ir-memory-map-basic
+
----------------
The only difference in this test is the command line used to compile the 
inferior right? That sounds like something that we will run into for a lot of 
lit test, so I think it's important to work something our right away. Making a 
windows-flavoured copy of each test is not tractable.

Is there a reason you have to use clang-cl here? I was under the impression 
that clang.exe worked fine on windows too (and used a gcc-compatible command 
line)...


https://reviews.llvm.org/D52618



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to