[PATCH] D16762: llvm-config: Add preliminary Windows support

NAKAMURA Takumi via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 9 17:10:46 PST 2016


chapuni added a comment.

Sorry, I misunderstood that the behavior weren't just for MSVC host. It is better.
That said, I suggest to keep behavior, just for the separator, by default. (lib's prefix and sufiix should be fine)


================
Comment at: llvm/trunk/tools/llvm-config/llvm-config.cpp:364
@@ +363,3 @@
+      StaticExt = "lib";
+      DirSep = "\\";
+      std::replace(ActiveObjRoot.begin(), ActiveObjRoot.end(), '/', '\\');
----------------
It confuses cross compilation, due to r252532. http://reviews.llvm.org/D13198

I will fix it later.


Repository:
  rL LLVM

http://reviews.llvm.org/D16762





More information about the llvm-commits mailing list