[PATCH] D43239: Improve internal StringSwitch code gen with Optional<T>

Doug Gregor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 13 12:07:01 PST 2018


doug.gregor accepted this revision.
doug.gregor added a comment.
This revision is now accepted and ready to land.

Heh, I was probably thinking that one might want to produce references, but I don’t anyone does. I do think it would be valuable to make the parameters “T value” instead of “const T value”, and std::move them into place when we find a match.


Repository:
  rL LLVM

https://reviews.llvm.org/D43239





More information about the llvm-commits mailing list