Fix unreachable-break issue in cea/chips/cache_benchmarks/memtier_client/protocol.cpp +5

Summary:
LLVM has a warning `-Wunreachable-code-break` which identifies `break` statements that cannot be reached. These compromise readability, are misleading, and may identify bugs. This diff removes such statements.

Such statements once existed to prevent accidental fallthroughs in switch statements. However, this is no longer necessary in C++17 because `[[fallthrough]]` is used to indicate intentional fallthroughs and we raise compilation errors for fallthroughs that are not annotated with `[[fallthrough]]` using `-Wimplicit-fallthrough`.

For questions/comments, contact rbarnes.

 - If you approve of this diff, please use the "Accept & Ship" button :-)

Reviewed By: dtolnay

Differential Revision: D78275946

fbshipit-source-id: 10be8205e8567233eac63b10a772d10987d3480f

Author: r-barnes

Jit/codegen/autogen.cpp

@@ -122,7 +122,6 @@ void AutoTranslator::translateInstr(Environ* env, const Instruction* instr)
         break;
       default:
         JIT_ABORT("Output operand has to be of type register or memory");
-        break;
     }
   }
 
@@ -144,7 +143,6 @@ void AutoTranslator::translateInstr(Environ* env, const Instruction* instr)
         break;
       default:
         JIT_ABORT("Illegal input type.");
-        break;
     }
   });
 
@@ -323,7 +321,6 @@ void TranslateCompare(Environ* env, const Instruction* instr) {
       break;
     default:
       JIT_ABORT("bad instruction for TranslateCompare");
-      break;
   }
   if (instr->output()->dataType() != OperandBase::k8bit) {
     as->movzx(