Re: [PATCH] D51847: Print correctly dependency paths on Windows

2018-09-12 Thread Zachary Turner via cfe-commits
Lgtm On Wed, Sep 12, 2018 at 3:23 AM Dávid Bolvanský via Phabricator < revi...@reviews.llvm.org> wrote: > xbolva00 updated this revision to Diff 165052. > xbolva00 added a comment. > > - Fixed failing test > > > https://reviews.llvm.org/D51847 > > Files: > lib/Frontend/DependencyFile.cpp > tes

Re: [PATCH] D51847: Print correctly dependency paths on Windows

2018-09-11 Thread Zachary Turner via cfe-commits
I mean in practice. What command do i run to hit this and what will the output look like? Can you paste some terminal output showing the command and output? On Tue, Sep 11, 2018 at 12:55 AM Dávid Bolvanský via Phabricator < revi...@reviews.llvm.org> wrote: > xbolva00 added a comment. > > In https: