efriedma added a reviewer: rsmith. efriedma added inline comments.
================ Comment at: lib/Sema/SemaChecking.cpp:228 + S.getASTContext(), Ty, /*consume*/ false); + Arg = S.PerformCopyInitialization(Entity, SourceLocation(), Arg); + TheCall->setArg(2, Arg.get()); ---------------- Is it possible for this initialization to fail? If it can't fail, please add an assertion; otherwise, you probably need to "return true". (Not sure off the top of my head how that would happen; maybe it would error out on a `volatile int*`?) Repository: rC Clang https://reviews.llvm.org/D48053 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits