[PATCH] D87808: [DebugInfo] Fix bug in constructor homing where it would use ctor homing when a class only has copy/move constructors
David Blaikie via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 23 19:29:56 PDT 2020
dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.
Generally looks good to me!
================
Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:2287
+ // don't have trivial or constexpr constructors, or can be created from
+ // aggregate initialization. Also skip lambda objects.
+
----------------
Maybe flesh out the "lambda objects" bit - something about how they are constructed without calling/having a constructor, I guess?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87808/new/
https://reviews.llvm.org/D87808
More information about the cfe-commits
mailing list