https://github.com/ilovepi approved this pull request.

Since this just moves common code I think this is fine for now. We should 
revisit some of the interfaces though, especially the ones that return 
SmallString<128>.  I think there is also a Path API that handles relative 
paths, so we can perhaps drop that entirely?

Also I know I suggested File.cpp, but maybe FileUtils.cpp is a more apt name? 
Now that I look at the content again?

https://github.com/llvm/llvm-project/pull/134298
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to