[llvm-dev] Passing Twine by value

[David Blaikie via llvm-dev]

I don't believe there's any correctness issue with passing Twine by value.

Not sure it'd even be a performance problem - the sizeof(Twine) looks like
it'd be small enough (like StringRef - though a bit bigger, StringRef
should be two words, Twine would be three) not to worry too much either way.


On Thu, May 30, 2019 at 12:14 AM Shoaib Meenai via llvm-dev <
llvm-dev at lists.llvm.org> wrote:

> Twine.h says twines should only be accepted as const references in
> arguments, but there are a bunch of functions that take Twine by value
> (e.g. you'll find lots of them in CGObjCMac.cpp in clang). Is this just a
> potential performance issue, or are there potential correctness issues from
> passing a twine by value as well?
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20190530/ec530054/attachment.html>