yaxunl added a comment. In D102270#2751852 <https://reviews.llvm.org/D102270#2751852>, @tra wrote:
> LGTM in general. > Perhaps it would make sense to combine this patch with D102237 > <https://reviews.llvm.org/D102237> as both patches are changing the same code > for the same reason, just for slightly different kinds of variables. Will merge D102237 <https://reviews.llvm.org/D102237> into this one. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102270/new/ https://reviews.llvm.org/D102270 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
