[PATCH] D71991: Fix external-names.c test when separator is \\

Adrian McCarthy via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 30 16:22:17 PST 2019


amccarth added a comment.

I'm still not seeing how the backslashes got in there for you.  They don't happen for me.  It looks like these paths come (almost) directly from the temp .yaml files created by the `sed` commands at the beginning of the test, so--for these particular tests--I'd expect forward slashes regardless of platform.

I made a set of interelated VFS changes for Windows compatibility.  The last of those was on December 18.  Could your repo be older than that?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71991/new/

https://reviews.llvm.org/D71991





More information about the cfe-commits mailing list