<div dir="ltr"><div>This is an obvious case.<br></div>Here is the original code flow:<br><br>switch (Name[0]) {<br>...<br>  case 'n': {<br>    if (Name.startswith("nvvm.")) {<br>    ...<br>    }<br>  }<br>  case 'o':<br>    ...<br><div>    if (Name.startswith("objectsize.")) {<br>    ...<br>    }<br>    break;<br>...<br><br></div><div>If the Name starts with 'n' but falls through to the next case, the expression in the second if is always false, so it always hit the second break after it falls through.<br><br></div><div>There is no functional change.<br><br></div><div>Thanks<br><br></div><div>Galina<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 19, 2017 at 1:50 PM, Davide Italiano <span dir="ltr"><<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, May 19, 2017 at 1:31 PM, Galina Kistanova via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
> Author: gkistanova<br>
> Date: Fri May 19 15:31:51 2017<br>
> New Revision: 303454<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=303454&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=303454&view=rev</a><br>
> Log:<br>
> Added missing break.<br>
><br>
<br>
</span>This seems like a functional change. I'm afraid it's going to be very<br>
hard to test this particular change in behaviour, but, can you please<br>
try?<br>
(Also, I assume this change is because you're trying to silence some<br>
warning of sort). It will be good to specify that in the commit<br>
(including the warning itself, maybe).<br>
<br>
Thanks,<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Davide<br>
<br>
"There are no solved problems; there are only problems that are more<br>
or less solved" -- Henri Poincare<br>
</font></span></blockquote></div><br></div></div>