[PATCH] D24882: Add StringSwitch::Cases functions that takes 6, 7 or 8 arguments.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 2 21:15:59 PDT 2017


davide added a comment.

In https://reviews.llvm.org/D24882#859796, @davide wrote:

> I ended up in a situation where I needed 11 arguments (if you want a similar example, just grep for `StringSwitch<bool>` in MC).
>  I think the proposed solution was OK'ish up to 10 arguments, but adding more might be not ideal. So, now that we live in a post C++98 world, what do you think about switching this class to use variadic templating? I'm going to implement such a solution if nobody has objections. (cc: @dblaikie for thoughts)


See also https://reviews.llvm.org/D37420


Repository:
  rL LLVM

https://reviews.llvm.org/D24882





More information about the llvm-commits mailing list