[PATCH] D26696: Object: replace backslashes with slashes in embedded relative thin archive paths on Windows.
George Rimar via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 16 00:02:07 PST 2016
grimar added a comment.
In https://reviews.llvm.org/D26696#597063, @davide wrote:
> In https://reviews.llvm.org/D26696#597061, @grimar wrote:
>
> > We had similar change in LLD, but did not use LLVM_ON_WIN32 define. May be it can be eliminated ?
>
>
> There's no point in replacing on *NIX as `/` is naturally the path separator there.
Yeah, I know. I mean that just helps to avoid use of define. Was just a suggestion, since we do in that way in LLD.
Repository:
rL LLVM
https://reviews.llvm.org/D26696
More information about the llvm-commits
mailing list