[PATCH] D70302: [CodeGen] Fix clang crash on aggregate initialization of array of labels

Johannes Altmanninger via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 15 08:18:37 PST 2019


johannes marked an inline comment as done.
johannes added inline comments.


================
Comment at: clang/lib/CodeGen/CGExpr.cpp:1021
 void CodeGenModule::EmitExplicitCastExprType(const ExplicitCastExpr *E,
-                                             CodeGenFunction *CGF) {
+                                             CodeGenFunction *const CGF) {
   // Bind VLAs in the cast type.
----------------
nickie wrote:
> I don't think that `const` is necessary here.  We're talking about the function's parameter.  Why isn't `E` a `*const` too?
Right, this was left over from when I thought it was necessary. Removed the `const` because it is surprising.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70302





More information about the cfe-commits mailing list