This is an archive of the discontinued LLVM Phabricator instance.

Remove not needed call to __is_long()
ClosedPublic

Authored by nilayvaish on Nov 15 2021, 9:21 AM.

Details

Summary

The string is known to be long since grow_by unconditionally calls
set_long_cap().

Diff Detail

Event Timeline

nilayvaish created this revision.Nov 15 2021, 9:21 AM
nilayvaish published this revision for review.Nov 15 2021, 10:00 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 15 2021, 10:00 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Quuxplusone added a subscriber: Quuxplusone.

Seems right to me, as long as CI is green. (But wait for someone to "accept as libc++" before you land this.)
If you don't have commit rights, then someone else will have to land it; in that case please provide your full name and email address, here.

ldionne accepted this revision.Nov 16 2021, 8:25 AM

Will edit as-of committing.

libcxx/include/string
2661
This revision is now accepted and ready to land.Nov 16 2021, 8:25 AM
This revision was automatically updated to reflect the committed changes.