[PATCH] D98251: [-Wcompletion-handler] Extend list of detected conventions

[Valeriy Savchenko via Phabricator via cfe-commits]

vsavchenko added a comment.

In D98251#2615049 <https://reviews.llvm.org/D98251#2615049>, @NoQ wrote:

> 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?

You are right on that part, I'll try to incorporate that check into the current (a bit tangled) logic.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98251/new/

https://reviews.llvm.org/D98251