bruno added a reviewer: bruno. bruno added inline comments.
================ Comment at: lib/Driver/ToolChains.cpp:3915 - if (D.getVFS().exists("/etc/SuSE-release")) - return OpenSUSE; + File = llvm::MemoryBuffer::getFile("/etc/SuSE-release"); + if (File) ---------------- You should keep using the VFS to obtain the file. You probably also want to add a comment here to describe what you mentioned in the patch Summary. https://reviews.llvm.org/D24954 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits