ChuanqiXu added a comment. I think it might be necessary/better to add a test for this. Otherwise, it looks not good to change this.
> This fixes the assertion for a downstream case in ROOT/Cling with the > involvement of modules. If anyone has ideas how to test this, please let me > know... Yeah, I know, this is pretty hard. And it would generally take a lot of time doing it. The method I used is to: - preprocess the failure files - Remove the unnecessary information, like a lot of line markers. - reduce them - repeating remove some part of the codes and see if the crash exists - Rename variables (if need) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137787/new/ https://reviews.llvm.org/D137787 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
