frasercrmck added a comment. In D106939#2915784 <https://reviews.llvm.org/D106939#2915784>, @HsiangKai wrote:
> In D106939#2915134 <https://reviews.llvm.org/D106939#2915134>, @HsiangKai > wrote: > >> In D106939#2912807 <https://reviews.llvm.org/D106939#2912807>, @frasercrmck >> wrote: >> >>> LGTM but there are test failures. Is that just a whole load of `mu->ma` >>> changes that have been omitted for a smaller diff? >> >> Updated test cases are put in https://reviews.llvm.org/D107022. > > Should I combine these two into one patch? Hmm, unsure, the test changes are pretty much unreviewable given how large the diff would be. There's nothing unexpected in there, presumably? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106939/new/ https://reviews.llvm.org/D106939 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
