dexonsmith marked 2 inline comments as done. dexonsmith added inline comments.
================ Comment at: clang/include/clang/Basic/FileManager.h:320 + /// twice, you get two new file entries. + const FileEntry *getBypassFile(const FileEntry &VFE); + ---------------- bruno wrote: > Does it make sense to return/read from a FileEntryRef here? @arphaman, wdyt? SGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66710/new/ https://reviews.llvm.org/D66710 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits