[PATCH] D43436: [Perf] Simplify llvm::StringSwitch and improve incremental rebuild by 54%

David Zarzycki via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 26 06:01:49 PST 2018


davezarzycki updated this revision to Diff 135889.
davezarzycki added a reviewer: jordan_rose.
davezarzycki added a comment.

I think this final diff addresses all concerns, including some amount of move-only type compatibility (it compiles, but I'm not sure the experience is pleasurable).

Jordan -- Chris recently gave me commit-after-approval access. Might you please approve this? Thanks!


Repository:
  rL LLVM

https://reviews.llvm.org/D43436

Files:
  include/llvm/ADT/StringRef.h
  include/llvm/ADT/StringSwitch.h
  unittests/ADT/StringSwitchTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43436.135889.patch
Type: text/x-patch
Size: 13267 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180226/9efde88b/attachment.bin>


More information about the llvm-commits mailing list