[Lldb-commits] [PATCH] D86660: Modifying ImportDeclContext(...) to ensure that we also handle the case when the FieldDecl is an ArrayType whose ElementType is a RecordDecl

Gabor Marton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Sep 17 00:10:19 PDT 2020


martong accepted this revision.
martong added a comment.
This revision is now accepted and ready to land.

In D86660#2278025 <https://reviews.llvm.org/D86660#2278025>, @shafik wrote:

> We are going to move forward with this approach (after dealing with the multi-dimensional array case) temporarily. We are seeing crash bugs from this from users and we want to fix it while we explore the solution space more.

Okay, I understand the business reasons and I am fine with this patch if this is really a temporary solution. Just please add a FIXME that indicates this and D71378 <https://reviews.llvm.org/D71378> are temporary, plus that this should be really handled by the preceding `import` call.


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

https://reviews.llvm.org/D86660



More information about the lldb-commits mailing list