This is an archive of the discontinued LLVM Phabricator instance.

[clang] Remove `DirectoryEntry::getName()`
Needs ReviewPublic

Authored by jansvoboda11 on Jun 1 2023, 1:50 PM.

Details

Summary

This patch finally removes the deprecated DirectoryEntry::getName() function.

Depends on D151927.

Diff Detail

Event Timeline

jansvoboda11 created this revision.Jun 1 2023, 1:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 1 2023, 1:50 PM
Herald added a subscriber: ributzka. · View Herald Transcript
jansvoboda11 requested review of this revision.Jun 1 2023, 1:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 1 2023, 1:50 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript

Given this was a commonly used function before and there's a decent chance it's used out of tree somewhere, should we wait until the next llvm release branch has been cut before landing this?

Yes, I'd like to get the deprecation (and fixes in other LLVM sub-projects) into main ASAP and then land this patch after the branch for the 17 release is cut.