sammccall added a subscriber: erik.pilkington. sammccall added a comment. The fix doesn't look obviously correct: the side effect of marking the destructor reference seems important if we actually generate code. It's not obvious to me why the type can only be incomplete if there are errors.
This was introduced between clang 8 and clang 9, I would guess by f8ccf052935adaf405e581fd31e8bc634cc5bbc7. @erik.pilkington Looking at that patch, mostly this function was just a rename, but there's a new callsite for array-types that seems to be what we're hitting here. Maybe a slightly less-invasive version would be to guard that callsite with "and if the element type is complete"? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99165/new/ https://reviews.llvm.org/D99165 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
