This revision was automatically updated to reflect the committed changes.
Closed by commit rL367618: [clang] Change FileManager to use llvm::ErrorOr
instead of null on failure (authored by harlanhaskins, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Change
harlanhaskins marked 6 inline comments as done and an inline comment as not
done.
harlanhaskins added inline comments.
Comment at: clang/lib/ARCMigrate/FileRemapper.cpp:156
+ auto newE = FileMgr->getFile(tempPath);
+ if (newE) {
+remap(origFE, *newE);
-
harlanhaskins updated this revision to Diff 212860.
harlanhaskins marked 8 inline comments as done.
harlanhaskins added a comment.
Updated in response to feedback
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65534/new/
https://reviews.llvm.org/D65
harlanhaskins updated this revision to Diff 212701.
harlanhaskins added a comment.
Store references instead of raw pointers in FileManger's cache
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65534/new/
https://reviews.llvm.org/D65534
Files:
cla
harlanhaskins added inline comments.
Comment at: clang/include/clang/Basic/FileManager.h:217
///
- /// This returns NULL if the file doesn't exist.
+ /// This returns a \c std::error_code if there was an error loading the file.
///
JDevlieghere wrote:
> h
harlanhaskins marked 2 inline comments as done.
harlanhaskins added inline comments.
Comment at: clang/include/clang/Basic/FileManager.h:143
///
- llvm::StringMap SeenDirEntries;
+ llvm::StringMap, llvm::BumpPtrAllocator>
+ SeenDirEntries;
jkorous wrote:
>
harlanhaskins created this revision.
harlanhaskins added reviewers: arphaman, bruno.
Herald added subscribers: lldb-commits, cfe-commits, jsji, kadircet,
dexonsmith, jkorous, MaskRay, kbarton, nemanjai.
Herald added a reviewer: martong.
Herald added a reviewer: shafik.
Herald added projects: clang
