dexonsmith added inline comments.
================ Comment at: clang/include/clang/Lex/Lexer.h:145 /// outlive it, so it doesn't take ownership of either of them. - Lexer(FileID FID, const llvm::MemoryBuffer *InputFile, Preprocessor &PP); + Lexer(FileID FID, const llvm::MemoryBufferRef &InputFile, Preprocessor &PP); ---------------- dexonsmith wrote: > JDevlieghere wrote: > > Should `MemoryBufferRef`s be passed by value like `StringRef`s? > I don't think it absolutely needs to be, but it could, it's just two > `StringRef`s. Passing by reference here prevents adding an include > unnecessarily. That said, I don't feel strongly, happy to change to by-value if you think that's better. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89398/new/ https://reviews.llvm.org/D89398 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
