This is an archive of the discontinued LLVM Phabricator instance.

[ADT] Deprecate StringRef::{starts,ends}with_insensitive
ClosedPublic

Authored by kazu on Jun 4 2023, 10:25 AM.

Details

Summary

This patch deprecates StringRef::{starts,ends}with_insensitive as
their uses have migrated to {starts,ends}_with_insensitive,
respectively.

Diff Detail

Event Timeline

kazu created this revision.Jun 4 2023, 10:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 4 2023, 10:25 AM
kazu requested review of this revision.Jun 4 2023, 10:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 4 2023, 10:25 AM
This revision is now accepted and ready to land.Jun 5 2023, 10:41 AM
bkramer accepted this revision.Jun 5 2023, 1:02 PM
This revision was automatically updated to reflect the committed changes.

Do we need to deprecate, or can we simply remove the old definitions?

Do we need to deprecate, or can we simply remove the old definitions?

startswith_insensitive appears to be used rare enough, mostly used within llvm-project. I think we can just remove them.