[llvm] [DecoderEmitter] Eliminate `DecodeComplete` parameter to `decodeToMCInst` (PR #159130)

Sergei Barannikov via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 16 12:11:12 PDT 2025


================
@@ -1069,9 +1065,7 @@ static std::string getDecoderString(const InstructionEncoding &Encoding) {
   StringRef DecoderMethod = Encoding.getDecoderMethod();
   if (!DecoderMethod.empty()) {
     OS << Indent << "if (!Check(S, " << DecoderMethod
-       << "(MI, insn, Address, Decoder))) { "
-       << (Encoding.hasCompleteDecoder() ? "" : "DecodeComplete = false; ")
-       << "return MCDisassembler::Fail; }\n";
+       << "(MI, insn, Address, Decoder))) { return MCDisassembler::Fail; }\n";
----------------
s-barannikov wrote:

Drop the braces? They were needed before because there could be two statements inside them.
Same above.

https://github.com/llvm/llvm-project/pull/159130


More information about the llvm-commits mailing list