[PATCH] D131555: [Clang] Propagate const context info when emitting compound literal

[Victor Campos via Phabricator via cfe-commits]

vhscampos added inline comments.


================
Comment at: clang/lib/CodeGen/CGExprConstant.cpp:2217
   assert(E->isFileScope() && "not a file-scope compound literal expr");
-  return tryEmitGlobalCompoundLiteral(*this, nullptr, E);
+  ConstantEmitter emitter(*this, nullptr);
+  return tryEmitGlobalCompoundLiteral(emitter, E);
----------------
vhscampos wrote:
> This constructor has the second parameter optional anyway. I suggest you omit the nullptr here.
To clarify, when I say optional, I mean it has a default value with is already nullptr


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131555