[PATCH] D50948: lambdas in modules: change storage for LambdaDefinitionData

[Richard Smith - zygoloid via Phabricator via cfe-commits]

rsmith added a comment.

We don't want to allocate storage for the lambda fields for non-lambda classes, which is why we use distinct base classes. Is the problem you're trying to solve here that we fake a definition in AST deserialization before we know whether the class is a lambda? If so, that seems solvable by moving the IsLambda flag out of DefinitionData into CXXRecordData (perhaps as a distinct TagTypeKind?).


Repository:
  rC Clang

https://reviews.llvm.org/D50948