[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