<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">@@ -666,6 +666,7 @@<br>
        default:<br>
            assert(false && "Unhandled Opcode.");<br>
     }<br>
+    return static_cast<LLVMOpcode>(0);<br></blockquote><div><br></div><div>If you used llvm_unreachable instead of assert(false) in that default case, then you should be able to go without the (effectively unreachable) return here.<br>
<br>And/or you could skip the default entirely if your switch covers all the values of the enum - I sent out a CR to remove all these unnecessary default cases (in both clang & llvm) a while back but I'm not sure anyone said 'yae' or 'nae' to it.</div>
<div><br></div><div>- David</div></div>