[PATCH] D51847: Print correctly dependency paths on Windows

Dávid Bolvanský via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 11 09:47:59 PDT 2018


xbolva00 added a comment.

In https://reviews.llvm.org/D51847#1230766, @zturner wrote:

> Alright I see it.  The reason I'm asking is because after your change you're now printing `main.o: main.c ../include/lib/test.h`.  But I wonder if you should instead be printing `main.o: main.c ..\include\lib\test.h`.  It seems like paths that we show to the user should be in the native path style.


Possibly, or should we follow GGC behavior here? GCC prints  `main.o: main.c ../include/lib/test.h`. But conversion to native paths sounds fine for me as well.


https://reviews.llvm.org/D51847





More information about the cfe-commits mailing list