nickdesaulniers added a comment.

In D156185#4530163 <https://reviews.llvm.org/D156185#4530163>, @efriedma wrote:

>> so if you were concerned with arrays of string literals, we need this patch.
>
> Ohhh.... that's not what I meant by StringLiterals.  I meant cases where the 
> string literal is an rvalue, like `char foo[100000] = "x";`.  If it's an 
> lvalue, we go through ConstantLValueEmitter, which is still reasonably fast.

This patch //is// addressing the case of the string literal rvalue; see the 
commit description:

>>> const char* foo = "foo";


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156185/new/

https://reviews.llvm.org/D156185

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

Reply via email to