MarcusJohnson91 added a comment. In D88084#2287450 <https://reviews.llvm.org/D88084#2287450>, @MyDeveloperDay wrote:
> I noticed the pre-merge tests failed! Yeah I just noticed that too, not sure what's up but I'll check into it, and yeah that's a good idea about initializing some of these duplicate variables in the constructor. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88084/new/ https://reviews.llvm.org/D88084 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits