<div dir="ltr">If nobody else have comments about this patch or thinks is not worth it is it possible to commit it?<div><br></div><div>I don't have commit access :)</div><div><br></div><div>Marcello</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">2014-06-12 0:22 GMT+01:00 Marcello Maggioni <span dir="ltr"><<a href="mailto:hayarms@gmail.com" target="_blank">hayarms@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div>I'm updating the patch to also remove the default basic block if it ends up being dead after switch lowering (no predecessors)<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Marcello<br>

<br><div class="gmail_quote">2014-06-11 20:34 GMT+01:00 Marcello Maggioni <span dir="ltr"><<a href="mailto:hayarms@gmail.com" target="_blank">hayarms@gmail.com</a>></span>:<div><div class="h5"><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr">Joerg,<div><br></div><div>I addressed your suggestion in this patch.</div><div>It was quite easy to add and can be useful in general, so thanks!</div><div><br></div><div>I also added a test that tests this kind of optimization being applied.</div>


<div><br></div><div>Marcello</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-06-11 18:37 GMT+01:00 Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de" target="_blank">joerg@britannica.bec.de</a>></span>:<div>

<div><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Wed, Jun 11, 2014 at 05:28:05PM +0100, Marcello Maggioni wrote:<br>
> Yeah, it is suboptimal, it doesn't take into consideration the fact that<br>
> the default is unreachable.<br>
><br>
> I'll a look at it later to see if it is easy to also take into<br>
> consideration this case and add it to the patch + test for the condition.<br>
<br>
</div>Thanks. Please don't take this as hold up for pushing the original<br>
patch, it can just as well be a follow-up commit.<br>
<div><div><br>
Joerg<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div></div></div><br></div>
</blockquote></div></div></div><br></div></div>
</blockquote></div><br></div>