[PATCH] D24686: [support] Some improvements to StringSwitch

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 3 07:31:09 PDT 2016


zturner added a comment.

Sadly, the variadic template approach died after Richard and I found out that it couldn't be made to work.  So all we've got now is the addition of `Lower` versions of the functions.  I just realized that the test file didn't make it to the review, so I'll get that uploaded and change to using `Optional`.  Do you think it's still worth splitting if we're not doing the variadic templates?


https://reviews.llvm.org/D24686





More information about the llvm-commits mailing list