[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