[PATCH] D65481: NFCI: Simplify SourceManager::translateFile by removing code path that should never be taken
Alex Lorenz via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 30 15:31:39 PDT 2019
arphaman created this revision.
arphaman added reviewers: rsmith, bruno, Bigcheese.
Herald added subscribers: dexonsmith, jkorous.
Herald added a project: clang.
I noticed that `SourceManager::translateFile` has code that doesn't really make sense. In particular, if it fails to find a `FileID` by comparing `FileEntry *` values, it tries to look through files that have the same filename, to see if they have a matching inode to try to find the right `FileID`. However, the inode comparison seem redundant, as Clang's `FileManager` already deduplicates `FileEntry *` values by inode. Thus the comparisons between inodes should never actually succeed, and the comparison between `FileEntry *` values should be sufficient here.
This observation is supported by the code coverage report that shows that we never actually reach the case where the INODE comparison succeeds:
rG LLVM Github Monorepo
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 6085 bytes
Desc: not available
More information about the cfe-commits