efriedma added inline comments.
================ Comment at: clang/lib/CodeGen/CGDecl.cpp:2459 + if (!isa<llvm::GlobalValue>(Arg.getAnyValue())) + Arg.getAnyValue()->setName(D.getName()); ---------------- rsmith wrote: > Is it feasible to only set the name if the value doesn't already have a name, > or do we give values bad names often enough that it's better to overwrite an > existing name here? Outside the EmitInlinedInheritingCXXConstructorCall case, the arguments are generally either llvm::Argument, or llvm::Instruction generated by calling convention code. It might make sense to just make the calling convention code name the values itself (it already names some of the values it generates). But I'll stick with the conservative fix here, and leave that for a followup. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154270/new/ https://reviews.llvm.org/D154270 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits