[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