[PATCH] D50966: Fix an undefined behavior when storing an empty StringRef.

[Joerg Sonnenberger via Phabricator via cfe-commits]

joerg added a comment.

Why do we need to allocate memory in this case at all? I.e. why can't this just 
be:

  if (S.empty())
    return StringRef("", 0);
  ...


Repository:
  rL LLVM

https://reviews.llvm.org/D50966



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits