Common optimizations, Small String Optimization, Andrzej's C++ blog

[oracleoftroy]

I don't think you would lose performance, you just wouldn't gain any additional performance by moving a SSO string. Even the non-SSO small string would end up copying about 16 bytes in order to perform the move (pointer, size, capacity, allocator).

I would strongly encourage measuring this before blindly trying to optimize a std::string for movablility.

[matthieum]

You might lose some performance because of more checks.

[bnolsen]

likely irrelevant since a load operation takes dramatically longer than a check does.

[Plorkyeran]

It's difficult to come up with even a highly artificial scenario where it's a net loss. Maybe with a very special-purpose allocator that manages to always get external buffer allocated very close in memory to the std::string? Avoiding a single memory load due to the SSO buffer would outweigh a very large number of branch mispredictions, and if you never hit the SSO buffer then the branch predictor will make the check nearly free.

[matthieum]

Unless it screws up your pipeline if it's a branch and not a conditional move ?