benlangmuir added inline comments.
================ Comment at: clang/lib/Basic/SourceManager.cpp:1016 + return *Name; return StringRef(); } ---------------- Just a suggestion: `value_or` can be a nice way to express simple cases like this: ``` getFilename(getFileID(SpellingLoc)).value_or(StringRef()); ``` ================ Comment at: clang/lib/Lex/PPLexerChange.cpp:136 PPCallbacks::EnterFile, FileType); + FileID PrevFID; + SourceLocation EnterLoc; ---------------- Why does `LexedFileChanged` have `PrevFID` set, but `FileChanged` does not (it has a default argument of `FileID()`? I would have expected that when you call both `FileChanged` and `LexedFileChanged` for the same event this would match. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128947/new/ https://reviews.llvm.org/D128947 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits