[llvm-commits] [PATCH 2/4] Add a copy constructor to StringRef.

Erick Tryzelaar idadesub at users.sourceforge.net
Thu Aug 13 01:18:08 PDT 2009


On Wed, Aug 12, 2009 at 8:55 PM, Daniel Dunbar <daniel at zuster.org> wrote:

> Hi Erick,
>
> In principle I'm fine with adding this, although I don't see why it is
> necessary. Since StringRef is immutable, it never makes sense to
> explicitly copy it?
>

Thanks for the reviews and commits! I originally added this because I
thought I was going to need it for implementing my changes to APInt and
APFloat, but then I guess I never actually used it. So you're right, it can
be safely ignored.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090813/0dc0a2ea/attachment.html>


More information about the llvm-commits mailing list