[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