[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