[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