[PATCH] Make getLastArgNoClaim work for up to 4 arguments.
Sean Silva
chisophugis at gmail.com
Mon Mar 23 16:27:16 PDT 2015
In http://reviews.llvm.org/D8508#144677, @probinson wrote:
> In http://reviews.llvm.org/D8508#144526, @Bigcheese wrote:
>
> > Why not make this a variadic template?
>
>
> A variadic "matches" method on Option would keep it from being
> amazingly inefficient. (i.e. don't iterate through ArgList N times.)
The real solution is to not have to walk the list for every method call like this. I.e. use a data structure that efficiently supports these sorts of queries.
REPOSITORY
rL LLVM
http://reviews.llvm.org/D8508
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list