[PATCH] D151587: [clang][ConstantEmitter] have tryEmitPrivateForVarInit try ConstExprEmitter fast-path first

[Nick Desaulniers via Phabricator via cfe-commits]

nickdesaulniers added a comment.

In D151587#4397446 <https://reviews.llvm.org/D151587#4397446>, @efriedma wrote:

> The following also crashes, with no MaterializeTemporaryExpr involved.
>
>   struct X {
>     short n;
>     char c;
>   };
>   
>   struct Y {
>     _Atomic(X) a;
>     _Atomic(int) b;
>   };
>   constexpr X x{};
>   int z;
>   Y y = { x, z };

Yeah, but not because of this patch; that's a pre-existing issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151587