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 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 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 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
This revision was automatically updated to reflect the committed changes.
Closed by commit rC358511: [FileSystemStatCache] Update test for new
FileSystemStatCache API (authored by harlanhaskins, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D60786?vs=195416&id=195417#toc
Re
harlanhaskins created this revision.
harlanhaskins added a reviewer: arphaman.
Herald added subscribers: cfe-commits, dexonsmith.
Herald added a project: clang.
arphaman accepted this revision.
arphaman added a comment.
This revision is now accepted and ready to land.
Herald added a subscriber: orm
This revision was automatically updated to reflect the committed changes.
Closed by commit rC358509: [FileSystemStatCache] Return std::error_code from
stat cache methods (authored by harlanhaskins, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D60735?vs=195264&id=195411#toc
harlanhaskins created this revision.
harlanhaskins added a reviewer: benlangmuir.
Herald added subscribers: cfe-commits, dexonsmith.
Herald added a project: clang.
Previously, we would return true/false signifying if the cache/lookup
succeeded or failed. Instead, provide clients with the underlyin
This revision was automatically updated to reflect the committed changes.
Closed by commit rL355368: Replace clang::FileData with llvm::vfs::Status
(authored by harlanhaskins, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https:/
harlanhaskins marked an inline comment as done.
harlanhaskins added inline comments.
Comment at: clang/lib/Basic/FileManager.cpp:305
+UFE = &UniqueRealFiles[Status.getUniqueID()];
+Status = llvm::vfs::Status(
+ Status.getName(), Status.getUniqueID(),
harlanhaskins created this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
harlanhaskins edited the summary of this revision.
harlanhaskins added a reviewer: benlangmuir.
Herald added a subscriber: ormris.
`FileData` was only ever used as a container for the values
14 matches
Mail list logo
