hokein marked an inline comment as done. hokein added inline comments.
================ Comment at: clang/lib/Sema/SemaInit.cpp:1640 expr = Result.getAs<Expr>(); // FIXME: Why are we updating the syntactic init list? + if (!VerifyOnly && expr) ---------------- I have the same question. but if you looked at the other 2 places (same FIXME) of this file, they reset the `Init` only when the `Result` is valid. I think this place maybe an overlook. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80980/new/ https://reviews.llvm.org/D80980 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
