Botond =?utf-8?q?István_Horváth?=,Botond Istvan Horvath <horvath.botond.ist...@gmail.com>,Botond Istvan Horvath <horvath.botond.ist...@gmail.com>,Botond Istvan Horvath <horvath.botond.ist...@gmail.com>,Botond Istvan Horvath <horvath.botond.ist...@gmail.com>,Botond Istvan Horvath <horvath.botond.ist...@gmail.com>,Botond Istvan Horvath <horvath.botond.ist...@gmail.com>,Botond Istvan Horvath <horvath.botond.ist...@gmail.com>,Botond Istvan Horvath <horvath.botond.ist...@gmail.com> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/83...@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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits