NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land.
Looks all clear. In that document there's also a clause that the conventional completion handler discovered through its suffix has to be the //last// parameter. I don't immediately see that respective check in the code. Do we have it? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98251/new/ https://reviews.llvm.org/D98251 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
