[clang] [clang] Bugfix for choosing the more specialized overload (PR #83279)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 5 12:13:15 PST 2024


Botond =?utf-8?q?István_Horváth?=,Botond Istvan Horvath
 <horvath.botond.istvan at gmail.com>,Botond Istvan Horvath
 <horvath.botond.istvan at gmail.com>,Botond Istvan Horvath
 <horvath.botond.istvan at gmail.com>,Botond Istvan Horvath
 <horvath.botond.istvan at gmail.com>,Botond Istvan Horvath
 <horvath.botond.istvan at gmail.com>,Botond Istvan Horvath
 <horvath.botond.istvan at gmail.com>,Botond Istvan Horvath
 <horvath.botond.istvan at gmail.com>,Botond Istvan Horvath
 <horvath.botond.istvan at gmail.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/83279 at github.com>


https://github.com/erichkeane approved this pull request.

Only some minor things left, so approving.  Mostly, for the 'type' and 'context', just do them the way they would read:
So "Object One Type" (plus we use Ty to mean type).  Same with Context.

Finally, `SmallVector` doesn't need a size set, so just using the 'default' value (which it calculates for you!)  is sufficient here.

Feel free to merge when CI agrees with you on the above/below changes.

https://github.com/llvm/llvm-project/pull/83279


More information about the cfe-commits mailing list