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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
