sammccall added a comment. In D140745#4508559 <https://reviews.llvm.org/D140745#4508559>, @nridge wrote:
> In D140745#4505829 <https://reviews.llvm.org/D140745#4505829>, @sammccall > wrote: > >> ping on this one for when you have time > > (Just wanted to double-check, is this ping directed to me or Kadir (or both)? > I haven't looked at this because Kadir has done a round of review and it > seemed like duplication of work for me to look at it as well; however, if > you'd like me to do a round of review, I'm happy to!) Oops, indeed it was directed at Kadir, we'd also discussed it offline at length a few months ago and I think this is mostly a question of cleaning up the details now. Always happy to have more feedback or hear concerns, but no need unless you're interested (and this patch isn't that interesting :-D) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140745/new/ https://reviews.llvm.org/D140745 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits