<div dir="ltr">FYI: this broke libFuzzer. <div>libFuzzer does not (and should not) include any of the LLVM headers, and so LLVM_FALLTHROUGH is not defined. <br></div><div>fixed in 278982</div><div><br></div><div>--kcc <br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 17, 2016 at 1:30 PM, Justin Bogner via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Author: bogner<br>
Date: Wed Aug 17 15:30:52 2016<br>
New Revision: 278970<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=278970&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=278970&view=rev</a><br>
Log:<br>
Replace a few more "fall through" comments with LLVM_FALLTHROUGH<br>
<br>
Follow up to r278902. I had missed "fall through", with a space.<br>
<br><br>
Modified: llvm/trunk/lib/Fuzzer/<wbr>FuzzerMutate.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Fuzzer/FuzzerMutate.cpp?rev=278970&r1=278969&r2=278970&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/llvm/trunk/lib/Fuzzer/<wbr>FuzzerMutate.cpp?rev=278970&<wbr>r1=278969&r2=278970&view=diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- llvm/trunk/lib/Fuzzer/<wbr>FuzzerMutate.cpp (original)<br>
+++ llvm/trunk/lib/Fuzzer/<wbr>FuzzerMutate.cpp Wed Aug 17 15:30:52 2016<br>
@@ -286,7 +286,7 @@ size_t MutationDispatcher::Mutate_<wbr>CrossO<br>
       NewSize = InsertPartOf(O.data(), O.size(), U.data(), U.size(), MaxSize);<br>
       if (NewSize)<br>
         break;<br>
-      // Fallthrough<br>
+      LLVM_FALLTHROUGH;<br>
     case 2:<br>
       NewSize = CopyPartOf(O.data(), O.size(), U.data(), U.size());<br>
       break;<br>
<br></blockquote></div></div></div></div>