ASDenysPetrov added a comment. @steakhal
> I would not accept this patch unless this investigation is done. However, I'm > not inherently against this change. Actually I've done the investigation. You can find it here https://reviews.llvm.org/D77062#1977613 The bug is somewhere deep in the core. I see two solutions: 1. Leave the crash presented in the clang and dig deep into the core trying to solve the root cause. 2. Accept the change with a TODO promt and dig deep into the core trying to solve the root cause. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77062/new/ https://reviews.llvm.org/D77062 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
