sammccall added a comment. Oops, forgot conclusion: Thanks for incorporating my testcase, and the patch LGTM in its current form (most of the alternatives discussed also sound OK). I'll leave to Haojian to stamp.
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121824/new/ https://reviews.llvm.org/D121824 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits