[PATCH] D75215: [DebugInfo] Fix for adding "returns cxx udt" option to functions in CodeView.

Amy Huang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 2 18:33:38 PST 2020


akhuang added a comment.

In D75215#1899224 <https://reviews.llvm.org/D75215#1899224>, @rnk wrote:

> Now that I've gone this far... maybe we should just say `isNonTrivial() || isFwdDecl()` -> mark it CxxReturnUdt. Something like that.


I just tried this but apparently it ends up marking things as CxxReturnUdt when the struct is trivial but not required to be complete--

So it seems like adding the NonTrivial flag to undefined forward declared functions would work.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75215/new/

https://reviews.llvm.org/D75215





More information about the llvm-commits mailing list