erichkeane accepted this revision. erichkeane added a comment. This revision is now accepted and ready to land.
In D121646#3380902 <https://reviews.llvm.org/D121646#3380902>, @royjacobson wrote: > In D121646#3380893 <https://reviews.llvm.org/D121646#3380893>, @erichkeane > wrote: > >> This seems acceptable to me. Though, I wonder if >> `getTemplateArgumentBindingsText` should produce the leading space, that way >> we could just pass the result of it directly, rather than have to create a >> SmallString everywhere. WDYT? > > There are other places (outside SemaOverload) that use this function and > don't need a space, though. > I thought about just putting the space in the diagnostic itself, but then > sometimes you don't get template arguments and it fails. I see. I was hoping that wasn't the case! OK, LGTM. Let me know if you need me to commit this for you. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121646/new/ https://reviews.llvm.org/D121646 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits