junparser added a comment. In D82314#2125713 <https://reviews.llvm.org/D82314#2125713>, @lxfind wrote:
> In D82314#2124662 <https://reviews.llvm.org/D82314#2124662>, @junparser wrote: > > > In D82314#2124661 <https://reviews.llvm.org/D82314#2124661>, @junparser > > wrote: > > > > > @lxfind This patch causes some mismatch when variable is used in both > > > resume and destroy function. Besides, we should move this patch and the > > > check in buildCoroutineFrame. > > > > > > @lxfind, Would you try to fix this? If you do not have time, then I'll try > > do this. Thanks > > > Could you please help take a look, if you have a local repro? Thanks! of course. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82314/new/ https://reviews.llvm.org/D82314 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits