This is an archive of the discontinued LLVM Phabricator instance.

Paths

[clang] NFCI: Use `DirectoryEntryRef` in framework lookup
Needs ReviewPublic

Authored by frs513 on Jun 21 2023, 4:17 PM.

This revision needs review, but there are no reviewers specified.

Summary

This removes one use of the deprecated DirectoryEntry::getName().

frs513 requested review of this revision.Jun 21 2023, 4:17 PM

Path

Size

clang/

lib/

Lex/

2 lines

Show First 20 Lines • Show All 684 Lines • ▼ Show 20 Lines	if (!File) {

File = FileMgr.getOptionalFileRef(FrameworkName,		File = FileMgr.getOptionalFileRef(FrameworkName,
/OpenFile=/!SuggestedModule);		/OpenFile=/!SuggestedModule);
}		}

// If we found the header and are allowed to suggest a module, do so now.		// If we found the header and are allowed to suggest a module, do so now.
if (File && needModuleLookup(RequestingModule, SuggestedModule)) {		if (File && needModuleLookup(RequestingModule, SuggestedModule)) {
// Find the framework in which this header occurs.		// Find the framework in which this header occurs.
StringRef FrameworkPath = File->getFileEntry().getDir()->getName();		StringRef FrameworkPath = File->getDir().getName();
bool FoundFramework = false;		bool FoundFramework = false;
do {		do {
// Determine whether this directory exists.		// Determine whether this directory exists.
auto Dir = FileMgr.getDirectory(FrameworkPath);		auto Dir = FileMgr.getDirectory(FrameworkPath);
if (!Dir)		if (!Dir)
break;		break;

// If this is a framework directory, then we're a subframework of this		// If this is a framework directory, then we're a subframework of this
▲ Show 20 Lines • Show All 1,337 Lines • Show Last 20 Lines