[PATCH] D41976: Low-hanging fruit optimization in string::__move_assign().
Timothy VanSlyke via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 16 07:13:58 PST 2018
tvanslyke added a comment.
I think the only thing we need is a call to __set_long_cap(0) in the body of the if() to make the string be in a valid state, but if you follow the logic for a call to reserve(0) (after having called clear()) it doesn't even end up doing that (unless I'm missing something). I definitely have no problem adding the __set_long_cap(0) call to the diff.
https://reviews.llvm.org/D41976
More information about the cfe-commits
mailing list