ziqingluo-90 added a comment. In D128401#3656559 <https://reviews.llvm.org/D128401#3656559>, @NoQ wrote:
> Looks great, thanks! > > Yes, I think refactoring can be done in a follow-up patch. Thank you, @NoQ! Let me politely ping the rest of the commenters: @LegalizeAdulthood, @gribozavr2, @Eugene.Zelenko. Is this patch looking good to you? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128401/new/ https://reviews.llvm.org/D128401 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits