[Lldb-commits] [PATCH] D115062: [LLDB][Clang] add AccessSpecDecl for methods and fields in RecordType

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

I like this feature, but I gotta say that a map feels a bit wasteful as well, given that the last access specifier is only needed for the short duration of time while the class definition is being parsed.

Maybe if you remove the relevant entry when `CompleteTagDeclarationDefinition` gets called, then we wouldn't at least have an unboundedly growing map on our hands?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115062