[PATCH] D30231: Replace backslashes with forward slashes

Galina via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 15:39:47 PST 2017


gkistanova added a comment.

Hi Zachary,

Unfortunately, this wouldn't work as you expect it to. The back slashes come from the `%(workdir)s` which gets rendered far away from this point.

The right fix should be in cmake to normalize the input paths by converting it with `file(TO_CMAKE_PATH,...)`.

Anyway, it seems we were looking at the same problem at the same time. I have committed a workaround for this issue in https://reviews.llvm.org/rL295785. Hope that will let you go further with your builder.


https://reviews.llvm.org/D30231





More information about the llvm-commits mailing list