[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
Tue Feb 20 08:55:38 PST 2018


davezarzycki updated this revision to Diff 135070.
davezarzycki edited the summary of this revision.
davezarzycki added a comment.

Sorry for the STRING_SWITCH proposal/noise. I was working on that in parallel with the templated version fixes. Now that I'm comparing the code gen of the two, I'm impressed with the optimizer generating essentially the STRING_SWITCH code gen. I'll keep the macro version in my back pocket in case I find a scenario where the optimizer fails to optimize the templated version code gen.


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.135070.patch
Type: text/x-patch
Size: 12800 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180220/e7537503/attachment.bin>


More information about the llvm-commits mailing list