[PATCH] LowerSwitch: replace unreachable default with popular case destination

Hans Wennborg hans at chromium.org
Tue Jan 20 17:39:40 PST 2015


> Yeah, that sounds like it!

>  This is exactly what I meant :)


Great! I've tidied up the patch a little, and made sure to use signed integers for IntRange since we're treating the case values as signed in the rest of the code.

Please take a look when you have time.


http://reviews.llvm.org/D6697

Files:
  lib/Transforms/Utils/LowerSwitch.cpp
  test/Transforms/LowerSwitch/2014-06-11-SwitchDefaultUnreachableOpt.ll
  test/Transforms/LowerSwitch/fold-popular-case-to-unreachable-default.ll

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6697.18470.patch
Type: text/x-patch
Size: 14009 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150121/d50f50a5/attachment.bin>


More information about the llvm-commits mailing list