[llvm-commits] [llvm] r137830 - in /llvm/trunk: include/llvm/MC/ lib/Target/ARM/Disassembler/ lib/Target/MBlaze/Disassembler/ lib/Target/X86/Disassembler/ test/MC/Disassembler/ARM/ tools/llvm-mc/ utils/TableGen/
James Molloy
james.molloy at arm.com
Thu Sep 1 08:56:31 PDT 2011
Hi Jim,
You didn't reply to my previous email, so I hope the attached patch is OK.
It addresses your issues in the following ways:
* Migrate MCDisassembler and EDDisassembler to use a switch and check for
SoftFail.
* SoftFail does the same as Fail at the moment, with a FIXME to possibly do
something more "nice".
* Makes early exit code explicit again, at the expense of some verbosity.
* Changes the macro call "CHECK" to a function "Check", which while slower is
explicit in its logic.
Is it OK to commit?
Cheers,
James
> -----Original Message-----
> From: llvm-commits-bounces at cs.uiuc.edu [mailto:llvm-commits-
> bounces at cs.uiuc.edu] On Behalf Of James Molloy
> Sent: 31 August 2011 12:07
> To: 'Jim Grosbach'
> Cc: llvm-commits LLVM
> Subject: Re: [llvm-commits] [llvm] r137830 - in /llvm/trunk:
> include/llvm/MC/ lib/Target/ARM/Disassembler/
> lib/Target/MBlaze/Disassembler/ lib/Target/X86/Disassembler/
> test/MC/Disassembler/ARM/ tools/llvm-mc/ utils/TableGen/
>
> Hi Jim,
>
> Apologies, somehow my spam filter got ahold of both of your replies. Lucky
> I
> checked it so soon after your ping, really :(
>
> Changing to switch() in Disassembler.cpp, need to change in MCDisassembler
> and
> EDDisassembler
> ---------------------------------------------------------------------------
> -----
> -------------
>
> I will do.
>
> Fail -> llvm::MCDisassembler::Fail
> ----------------------------------
>
> Can do; now that I've done the grunt work a simple regex will suffice to do
> that. I'd suggest "using namespace llvm;" though to shorten it slightly -
> are
> you OK with that?
>
> No early exit
> -------------
>
> I can remove this. I thought it made the code flow easier, but I see your
> point
> that hiding the early exit is inadvisable.
>
> Use enum values instead of &
> ----------------------------
>
> I implemented this as a binary-AND for two reasons:
> * Makes the macro, which is used everywhere, more compact and quicker
> than
> using if-statements.
> * Allows for more return types to be added if needed without any change
> to the
> code.
>
> If instead you'd prefer me to write the macro as a big switch:
>
> #define CHECK(S, X) do {
> switch ((X)) {
> case MCDisassembler::Success:
> break;
> case MCDisassembler::SoftFail:
> S = MCDisassembler::SoftFail;
> break;
> case MCDisassembler::Fail:
> S = MCDisassembler::Fail;
> default: llvm_unreachable();
> }
>
> I can, or better possibly as we're going the whole hog and not caring so
> much
> about comparison speed:
>
> bool Check(MCDisassembler::DecodeStatus &Out, MCDisassembler::DecodeStatus
> In) {
> switch (In) {
> case MCDisassembler::Success:
> return true;
> case MCDisassembler::SoftFail:
> Out = In;
> return true;
> case MCDisassembler::Fail:
> Out = In;
> return false;
> default: llvm_unreachable();
> }
> }
>
> Then the CHECK() lines could be changed to:
> if (!Check(S, ...)) return MCDisassembler::Fail;
>
> Which would satisfy your above criterion?
>
> Cheers,
>
> James
>
> > -----Original Message-----
> > From: Jim Grosbach [mailto:grosbach at apple.com]
> > Sent: 30 August 2011 20:15
> > To: James Molloy
> > Cc: llvm-commits LLVM; Owen Anderson
> > Subject: Re: [llvm-commits] [llvm] r137830 - in /llvm/trunk:
> > include/llvm/MC/ lib/Target/ARM/Disassembler/
> > lib/Target/MBlaze/Disassembler/ lib/Target/X86/Disassembler/
> > test/MC/Disassembler/ARM/ tools/llvm-mc/ utils/TableGen/
> >
> > ping?
> >
> > On Aug 26, 2011, at 11:56 AM, Jim Grosbach wrote:
> >
> > > Sorry for the late comments. I missed this catching these bits the
> first
> > time around (saw the patch, just missed this aspect).
> > >
> > > On Aug 17, 2011, at 10:44 AM, Owen Anderson wrote:
> > >
> > >> Author: resistor
> > >> Date: Wed Aug 17 12:44:15 2011
> > >> New Revision: 137830
> > >>
> > >> URL: http://llvm.org/viewvc/llvm-project?rev=137830&view=rev
> > >> Log:
> > >> Allow the MCDisassembler to return a "soft fail" status code,
> indicating
> > an instruction that is disassemblable, but invalid. Only used for ARM
> > UNPREDICTABLE instructions at the moment.
> > >> Patch by James Molloy.
> > >>
> > >> Modified:
> > >> llvm/trunk/include/llvm/MC/MCDisassembler.h
> > >> llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.cpp
> > >> llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.h
> > >> llvm/trunk/lib/Target/MBlaze/Disassembler/MBlazeDisassembler.cpp
> > >> llvm/trunk/lib/Target/MBlaze/Disassembler/MBlazeDisassembler.h
> > >> llvm/trunk/lib/Target/X86/Disassembler/X86Disassembler.cpp
> > >> llvm/trunk/lib/Target/X86/Disassembler/X86Disassembler.h
> > >> llvm/trunk/test/MC/Disassembler/ARM/invalid-LDRB_POST-arm.txt
> > >> llvm/trunk/tools/llvm-mc/Disassembler.cpp
> > >> llvm/trunk/utils/TableGen/DisassemblerEmitter.cpp
> > >> llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.cpp
> > >> llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.h
> > >>
> > >> Modified: llvm/trunk/include/llvm/MC/MCDisassembler.h
> > >> URL: http://llvm.org/viewvc/llvm-
> >
> project/llvm/trunk/include/llvm/MC/MCDisassembler.h?rev=137830&r1=137829&r2
> > =137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/include/llvm/MC/MCDisassembler.h (original)
> > >> +++ llvm/trunk/include/llvm/MC/MCDisassembler.h Wed Aug 17 12:44:15
> 2011
> > >> @@ -25,6 +25,34 @@
> > >> /// and provides an array of assembly instructions.
> > >> class MCDisassembler {
> > >> public:
> > >> + /// Ternary decode status. Most backends will just use Fail and
> > >> + /// Success, however some have a concept of an instruction with
> > >> + /// understandable semantics but which is architecturally
> > >> + /// incorrect. An example of this is ARM UNPREDICTABLE instructions
> > >> + /// which are disassemblable but cause undefined behaviour.
> > >> + ///
> > >> + /// Because it makes sense to disassemble these instructions, there
> > >> + /// is a "soft fail" failure mode that indicates the MCInst& is
> > >> + /// valid but architecturally incorrect.
> > >> + ///
> > >> + /// The enum numbers are deliberately chosen such that reduction
> > >> + /// from Success->SoftFail ->Fail can be done with a simple
> > >> + /// bitwise-AND:
> > >> + ///
> > >> + /// LEFT & TOP = | Success Unpredictable Fail
> > >> + /// --------------+-----------------------------------
> > >> + /// Success | Success Unpredictable Fail
> > >> + /// Unpredictable | Unpredictable Unpredictable Fail
> > >> + /// Fail | Fail Fail Fail
> > >> + ///
> > >> + /// An easy way of encoding this is as 0b11, 0b01, 0b00 for
> > >> + /// Success, SoftFail, Fail respectively.
> > >> + enum DecodeStatus {
> > >> + Fail = 0,
> > >> + SoftFail = 1,
> > >> + Success = 3
> > >> + };
> > >> +
> > >> /// Constructor - Performs initial setup for the disassembler.
> > >> MCDisassembler() : GetOpInfo(0), DisInfo(0), Ctx(0) {}
> > >>
> > >> @@ -41,8 +69,11 @@
> > >> /// @param address - The address, in the memory space of region, of
> > the first
> > >> /// byte of the instruction.
> > >> /// @param vStream - The stream to print warnings and diagnostic
> > messages on.
> > >> - /// @return - True if the instruction is valid; false
> > otherwise.
> > >> - virtual bool getInstruction(MCInst& instr,
> > >> + /// @return - MCDisassembler::Success if the instruction is
> > valid,
> > >> + /// MCDisassembler::SoftFail if the instruction
> was
> > >> + /// disassemblable but
> > invalid,
> > >> + /// MCDisassembler::Fail if the instruction was
> > invalid.
> > >> + virtual DecodeStatus getInstruction(MCInst& instr,
> > >> uint64_t& size,
> > >> const MemoryObject ®ion,
> > >> uint64_t address,
> > >>
> > >> Modified: llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.cpp
> > >> URL: http://llvm.org/viewvc/llvm-
> >
> project/llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.cpp?rev=1378
> > 30&r1=137829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.cpp
> > (original)
> > >> +++ llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.cpp Wed Aug
> > 17 12:44:15 2011
> > >> @@ -24,188 +24,201 @@
> > >> #include "llvm/Support/ErrorHandling.h"
> > >> #include "llvm/Support/raw_ostream.h"
> > >>
> > >> +// Pull DecodeStatus and its enum values into the global namespace.
> > >> +typedef llvm::MCDisassembler::DecodeStatus DecodeStatus;
> > >> +#define Success llvm::MCDisassembler::Success
> > >> +#define Unpredictable llvm::MCDisassembler::SoftFail
> > >> +#define Fail llvm::MCDisassembler::Fail
> > >> +
> > >
> > > Please don't do this sort of thing (#defines to get around scoping).
> > Reference the names explicitly, including the scoping operators, instead.
> > >
> > >> +// Helper macro to perform setwise reduction of the current running
> > status
> > >> +// and another status, and return if the new status is Fail.
> > >> +#define CHECK(S,X) do { \
> > >> + S = (DecodeStatus) ((int)S & (X)); \
> > >
> > > It's better to use the enum values directly rather than casting
> integers
> > like this. The code shouldn't know what the actual values of the enums
> are.
> > >
> > >> + if (S == Fail) return Fail; \
> > >> + } while(0)
> > >> +
> > >
> > > Having an early exit buried in a macro obfuscates the code. The way the
> > code previously did this is better. Please change this back or refactor
> > differently.
> > >
> > >> // Forward declare these because the autogenerated code will reference
> > them.
> > >> // Definitions are further down.
> > >> -static bool DecodeGPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecodeGPRRegisterClass(llvm::MCInst &Inst,
> unsigned
> > RegNo,
> > >> uint64_t Address, const void
> > *Decoder);
> > >> -static bool DecodeGPRnopcRegisterClass(llvm::MCInst &Inst, unsigned
> > RegNo,
> > >> +static DecodeStatus DecodeGPRnopcRegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> > *Decoder);
> > >> -static bool DecodetGPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecodetGPRRegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> > *Decoder);
> > >> -static bool DecodetcGPRRegisterClass(llvm::MCInst &Inst, unsigned
> > RegNo,
> > >> +static DecodeStatus DecodetcGPRRegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> > *Decoder);
> > >> -static bool DecoderGPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecoderGPRRegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> > *Decoder);
> > >> -static bool DecodeSPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecodeSPRRegisterClass(llvm::MCInst &Inst,
> unsigned
> > RegNo,
> > >> uint64_t Address, const void
> > *Decoder);
> > >> -static bool DecodeDPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecodeDPRRegisterClass(llvm::MCInst &Inst,
> unsigned
> > RegNo,
> > >> uint64_t Address, const void
> > *Decoder);
> > >> -static bool DecodeDPR_8RegisterClass(llvm::MCInst &Inst, unsigned
> > RegNo,
> > >> +static DecodeStatus DecodeDPR_8RegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> > *Decoder);
> > >> -static bool DecodeDPR_VFP2RegisterClass(llvm::MCInst &Inst, unsigned
> > RegNo,
> > >> +static DecodeStatus DecodeDPR_VFP2RegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> > *Decoder);
> > >> -static bool DecodeQPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecodeQPRRegisterClass(llvm::MCInst &Inst,
> unsigned
> > RegNo,
> > >> uint64_t Address, const void
> > *Decoder);
> > >>
> > >> -static bool DecodePredicateOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodePredicateOperand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeCCOutOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeCCOutOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeSOImmOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeSOImmOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeBLTargetOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeBLTargetOperand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeRegListOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeRegListOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeSPRRegListOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeSPRRegListOperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeDPRRegListOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeDPRRegListOperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >>
> > >> -static bool DecodeBitfieldMaskOperand(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> +static DecodeStatus DecodeBitfieldMaskOperand(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeCopMemInstruction(llvm::MCInst &Inst, unsigned
> Insn,
> > >> +static DecodeStatus DecodeCopMemInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeAddrMode2IdxInstruction(llvm::MCInst &Inst,
> unsigned
> > Insn,
> > >> +static DecodeStatus DecodeAddrMode2IdxInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeSORegMemOperand(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeSORegMemOperand(llvm::MCInst &Inst,
> unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeAddrMode3Instruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> +static DecodeStatus DecodeAddrMode3Instruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeSORegImmOperand(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeSORegImmOperand(llvm::MCInst &Inst,
> unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeSORegRegOperand(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeSORegRegOperand(llvm::MCInst &Inst,
> unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >>
> > >> -static bool DecodeMemMultipleWritebackInstruction(llvm::MCInst &
> Inst,
> > >> +static DecodeStatus
> DecodeMemMultipleWritebackInstruction(llvm::MCInst
> > & Inst,
> > >> unsigned Insn,
> > >> uint64_t Adddress,
> > >> const void *Decoder);
> > >> -static bool DecodeSMLAInstruction(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeSMLAInstruction(llvm::MCInst &Inst,
> unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeCPSInstruction(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeCPSInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeAddrModeImm12Operand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> +static DecodeStatus DecodeAddrModeImm12Operand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeAddrMode5Operand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeAddrMode5Operand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeAddrMode7Operand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeAddrMode7Operand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeBranchImmInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> +static DecodeStatus DecodeBranchImmInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVCVTImmOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeVCVTImmOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeAddrMode6Operand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeAddrMode6Operand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVLDInstruction(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeVLDInstruction(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVSTInstruction(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeVSTInstruction(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVLD1DupInstruction(llvm::MCInst &Inst, unsigned
> Val,
> > >> +static DecodeStatus DecodeVLD1DupInstruction(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVLD2DupInstruction(llvm::MCInst &Inst, unsigned
> Val,
> > >> +static DecodeStatus DecodeVLD2DupInstruction(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVLD3DupInstruction(llvm::MCInst &Inst, unsigned
> Val,
> > >> +static DecodeStatus DecodeVLD3DupInstruction(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVLD4DupInstruction(llvm::MCInst &Inst, unsigned
> Val,
> > >> +static DecodeStatus DecodeVLD4DupInstruction(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeNEONModImmInstruction(llvm::MCInst &Inst, unsigned
> > Val,
> > >> +static DecodeStatus DecodeNEONModImmInstruction(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVSHLMaxInstruction(llvm::MCInst &Inst, unsigned
> Val,
> > >> +static DecodeStatus DecodeVSHLMaxInstruction(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeShiftRight8Imm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeShiftRight8Imm(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeShiftRight16Imm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeShiftRight16Imm(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeShiftRight32Imm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeShiftRight32Imm(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeShiftRight64Imm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeShiftRight64Imm(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeTBLInstruction(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeTBLInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVFPfpImm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeVFPfpImm(llvm::MCInst &Inst, unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodePostIdxReg(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodePostIdxReg(llvm::MCInst &Inst, unsigned
> Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeCoprocessor(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeCoprocessor(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeMemBarrierOption(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeMemBarrierOption(llvm::MCInst &Inst,
> unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeMSRMask(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeMSRMask(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeDoubleRegLoad(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeDoubleRegLoad(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeDoubleRegStore(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeDoubleRegStore(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeSTRPreImm(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeSTRPreImm(llvm::MCInst &Inst, unsigned
> Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeSTRPreReg(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeSTRPreReg(llvm::MCInst &Inst, unsigned
> Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVLD1LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVLD1LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVLD2LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVLD2LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVLD3LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVLD3LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVLD4LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVLD4LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVST1LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVST1LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVST2LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVST2LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVST3LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVST3LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeVST4LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVST4LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder);
> > >>
> > >>
> > >> -static bool DecodeThumbAddSpecialReg(llvm::MCInst &Inst, uint16_t
> Insn,
> > >> +static DecodeStatus DecodeThumbAddSpecialReg(llvm::MCInst &Inst,
> > uint16_t Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbBROperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbBROperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeT2BROperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2BROperand(llvm::MCInst &Inst, unsigned
> Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbCmpBROperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbCmpBROperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbAddrModeRR(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbAddrModeRR(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbAddrModeIS(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbAddrModeIS(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbAddrModePC(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbAddrModePC(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbAddrModeSP(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbAddrModeSP(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeT2AddrModeSOReg(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2AddrModeSOReg(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeT2LoadShift(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2LoadShift(llvm::MCInst &Inst, unsigned
> Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeT2Imm8S4(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2Imm8S4(llvm::MCInst &Inst, unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeT2AddrModeImm8s4(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2AddrModeImm8s4(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeT2Imm8(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2Imm8(llvm::MCInst &Inst, unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeT2AddrModeImm8(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2AddrModeImm8(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbAddSPImm(llvm::MCInst &Inst, uint16_t Val,
> > >> +static DecodeStatus DecodeThumbAddSPImm(llvm::MCInst &Inst, uint16_t
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbAddSPReg(llvm::MCInst &Inst, uint16_t Insn,
> > >> +static DecodeStatus DecodeThumbAddSPReg(llvm::MCInst &Inst, uint16_t
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbCPS(llvm::MCInst &Inst, uint16_t Insn,
> > >> +static DecodeStatus DecodeThumbCPS(llvm::MCInst &Inst, uint16_t Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbBLXOffset(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeThumbBLXOffset(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeT2AddrModeImm12(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2AddrModeImm12(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbSRImm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbSRImm(llvm::MCInst &Inst, unsigned
> Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumb2BCCInstruction(llvm::MCInst &Inst, unsigned
> > Val,
> > >> +static DecodeStatus DecodeThumb2BCCInstruction(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeT2SOImm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2SOImm(llvm::MCInst &Inst, unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbBCCTargetOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> +static DecodeStatus DecodeThumbBCCTargetOperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >> -static bool DecodeThumbBLTargetOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> +static DecodeStatus DecodeThumbBLTargetOperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder);
> > >>
> > >> #include "ARMGenDisassemblerTables.inc"
> > >> @@ -230,15 +243,14 @@
> > >> return instInfoARM;
> > >> }
> > >>
> > >> -
> > >> -bool ARMDisassembler::getInstruction(MCInst &MI, uint64_t &Size,
> > >> - const MemoryObject &Region,
> > >> - uint64_t Address,raw_ostream
> &os)
> > const {
> > >> +DecodeStatus ARMDisassembler::getInstruction(MCInst &MI, uint64_t
> > &Size,
> > >> + const MemoryObject
> > &Region,
> > >> + uint64_t
> > Address,raw_ostream &os) const {
> > >> uint8_t bytes[4];
> > >>
> > >> // We want to read exactly 4 bytes of data.
> > >> if (Region.readBytes(Address, 4, (uint8_t*)bytes, NULL) == -1)
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> // Encoded as a small-endian 32-bit word in the stream.
> > >> uint32_t insn = (bytes[3] << 24) |
> > >> @@ -247,10 +259,10 @@
> > >> (bytes[0] << 0);
> > >>
> > >> // Calling the auto-generated decoder function.
> > >> - bool result = decodeARMInstruction32(MI, insn, Address, this);
> > >> - if (result) {
> > >> + DecodeStatus result = decodeARMInstruction32(MI, insn, Address,
> > this);
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> // Instructions that are shared between ARM and Thumb modes.
> > >> @@ -258,53 +270,53 @@
> > >> // fact that we fail to encode a few instructions properly for Thumb.
> > >> MI.clear();
> > >> result = decodeCommonInstruction32(MI, insn, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> // VFP and NEON instructions, similarly, are shared between ARM
> > >> // and Thumb modes.
> > >> MI.clear();
> > >> result = decodeVFPInstruction32(MI, insn, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> MI.clear();
> > >> result = decodeNEONDataInstruction32(MI, insn, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> // Add a fake predicate operand, because we share these instruction
> > >> // definitions with Thumb2 where these instructions are predicable.
> > >> - if (!DecodePredicateOperand(MI, 0xE, Address, this)) return
> false;
> > >> - return true;
> > >> + if (!DecodePredicateOperand(MI, 0xE, Address, this)) return Fail;
> > >> + return result;
> > >> }
> > >>
> > >> MI.clear();
> > >> result = decodeNEONLoadStoreInstruction32(MI, insn, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> // Add a fake predicate operand, because we share these instruction
> > >> // definitions with Thumb2 where these instructions are predicable.
> > >> - if (!DecodePredicateOperand(MI, 0xE, Address, this)) return
> false;
> > >> - return true;
> > >> + if (!DecodePredicateOperand(MI, 0xE, Address, this)) return Fail;
> > >> + return result;
> > >> }
> > >>
> > >> MI.clear();
> > >> result = decodeNEONDupInstruction32(MI, insn, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> // Add a fake predicate operand, because we share these instruction
> > >> // definitions with Thumb2 where these instructions are predicable.
> > >> - if (!DecodePredicateOperand(MI, 0xE, Address, this)) return
> false;
> > >> - return true;
> > >> + if (!DecodePredicateOperand(MI, 0xE, Address, this)) return Fail;
> > >> + return result;
> > >> }
> > >>
> > >> MI.clear();
> > >>
> > >> - return false;
> > >> + return Fail;
> > >> }
> > >>
> > >> namespace llvm {
> > >> @@ -403,22 +415,21 @@
> > >> }
> > >> }
> > >>
> > >> -
> > >> -bool ThumbDisassembler::getInstruction(MCInst &MI, uint64_t &Size,
> > >> - const MemoryObject &Region,
> > >> - uint64_t Address,raw_ostream
> > &os) const {
> > >> +DecodeStatus ThumbDisassembler::getInstruction(MCInst &MI, uint64_t
> > &Size,
> > >> + const MemoryObject
> > &Region,
> > >> + uint64_t
> > Address,raw_ostream &os) const {
> > >> uint8_t bytes[4];
> > >>
> > >> // We want to read exactly 2 bytes of data.
> > >> if (Region.readBytes(Address, 2, (uint8_t*)bytes, NULL) == -1)
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> uint16_t insn16 = (bytes[1] << 8) | bytes[0];
> > >> - bool result = decodeThumbInstruction16(MI, insn16, Address, this);
> > >> - if (result) {
> > >> + DecodeStatus result = decodeThumbInstruction16(MI, insn16, Address,
> > this);
> > >> + if (result != Fail) {
> > >> Size = 2;
> > >> AddThumbPredicate(MI);
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> MI.clear();
> > >> @@ -428,12 +439,12 @@
> > >> bool InITBlock = !ITBlock.empty();
> > >> AddThumbPredicate(MI);
> > >> AddThumb1SBit(MI, InITBlock);
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> MI.clear();
> > >> result = decodeThumb2Instruction16(MI, insn16, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 2;
> > >> AddThumbPredicate(MI);
> > >>
> > >> @@ -456,12 +467,12 @@
> > >> ITBlock.push_back(firstcond);
> > >> }
> > >>
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> // We want to read exactly 4 bytes of data.
> > >> if (Region.readBytes(Address, 4, (uint8_t*)bytes, NULL) == -1)
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> uint32_t insn32 = (bytes[3] << 8) |
> > >> (bytes[2] << 0) |
> > >> @@ -469,44 +480,44 @@
> > >> (bytes[0] << 16);
> > >> MI.clear();
> > >> result = decodeThumbInstruction32(MI, insn32, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> bool InITBlock = ITBlock.size();
> > >> AddThumbPredicate(MI);
> > >> AddThumb1SBit(MI, InITBlock);
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> MI.clear();
> > >> result = decodeThumb2Instruction32(MI, insn32, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> AddThumbPredicate(MI);
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> MI.clear();
> > >> result = decodeCommonInstruction32(MI, insn32, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> AddThumbPredicate(MI);
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> MI.clear();
> > >> result = decodeVFPInstruction32(MI, insn32, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> UpdateThumbVFPPredicate(MI);
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> MI.clear();
> > >> result = decodeNEONDupInstruction32(MI, insn32, Address, this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> AddThumbPredicate(MI);
> > >> - return true;
> > >> + return result;
> > >> }
> > >>
> > >> if (fieldFromInstruction32(insn32, 24, 8) == 0xF9) {
> > >> @@ -515,10 +526,10 @@
> > >> NEONLdStInsn &= 0xF0FFFFFF;
> > >> NEONLdStInsn |= 0x04000000;
> > >> result = decodeNEONLoadStoreInstruction32(MI, NEONLdStInsn,
> Address,
> > this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> AddThumbPredicate(MI);
> > >> - return true;
> > >> + return result;
> > >> }
> > >> }
> > >>
> > >> @@ -529,14 +540,14 @@
> > >> NEONDataInsn |= (NEONDataInsn & 0x10000000) >> 4; // Move bit 28 to
> > bit 24
> > >> NEONDataInsn |= 0x12000000; // Set bits 28 and 25
> > >> result = decodeNEONDataInstruction32(MI, NEONDataInsn, Address,
> > this);
> > >> - if (result) {
> > >> + if (result != Fail) {
> > >> Size = 4;
> > >> AddThumbPredicate(MI);
> > >> - return true;
> > >> + return result;
> > >> }
> > >> }
> > >>
> > >> - return false;
> > >> + return Fail;
> > >> }
> > >>
> > >>
> > >> @@ -554,30 +565,30 @@
> > >> ARM::R12, ARM::SP, ARM::LR, ARM::PC
> > >> };
> > >>
> > >> -static bool DecodeGPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecodeGPRRegisterClass(llvm::MCInst &Inst,
> unsigned
> > RegNo,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> if (RegNo > 15)
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> unsigned Register = GPRDecoderTable[RegNo];
> > >> Inst.addOperand(MCOperand::CreateReg(Register));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeGPRnopcRegisterClass(llvm::MCInst &Inst, unsigned
> > RegNo,
> > >> +static DecodeStatus DecodeGPRnopcRegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> - if (RegNo == 15) return false;
> > >> + if (RegNo == 15) return Fail;
> > >> return DecodeGPRRegisterClass(Inst, RegNo, Address, Decoder);
> > >> }
> > >>
> > >> -static bool DecodetGPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecodetGPRRegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> if (RegNo > 7)
> > >> - return false;
> > >> + return Fail;
> > >> return DecodeGPRRegisterClass(Inst, RegNo, Address, Decoder);
> > >> }
> > >>
> > >> -static bool DecodetcGPRRegisterClass(llvm::MCInst &Inst, unsigned
> > RegNo,
> > >> +static DecodeStatus DecodetcGPRRegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> unsigned Register = 0;
> > >> switch (RegNo) {
> > >> @@ -600,16 +611,16 @@
> > >> Register = ARM::R12;
> > >> break;
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> }
> > >>
> > >> Inst.addOperand(MCOperand::CreateReg(Register));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecoderGPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecoderGPRRegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> - if (RegNo == 13 || RegNo == 15) return false;
> > >> + if (RegNo == 13 || RegNo == 15) return Fail;
> > >> return DecodeGPRRegisterClass(Inst, RegNo, Address, Decoder);
> > >> }
> > >>
> > >> @@ -624,14 +635,14 @@
> > >> ARM::S28, ARM::S29, ARM::S30, ARM::S31
> > >> };
> > >>
> > >> -static bool DecodeSPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecodeSPRRegisterClass(llvm::MCInst &Inst,
> unsigned
> > RegNo,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> if (RegNo > 31)
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> unsigned Register = SPRDecoderTable[RegNo];
> > >> Inst.addOperand(MCOperand::CreateReg(Register));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> static const unsigned DPRDecoderTable[] = {
> > >> @@ -645,27 +656,27 @@
> > >> ARM::D28, ARM::D29, ARM::D30, ARM::D31
> > >> };
> > >>
> > >> -static bool DecodeDPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecodeDPRRegisterClass(llvm::MCInst &Inst,
> unsigned
> > RegNo,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> if (RegNo > 31)
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> unsigned Register = DPRDecoderTable[RegNo];
> > >> Inst.addOperand(MCOperand::CreateReg(Register));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeDPR_8RegisterClass(llvm::MCInst &Inst, unsigned
> > RegNo,
> > >> +static DecodeStatus DecodeDPR_8RegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> if (RegNo > 7)
> > >> - return false;
> > >> + return Fail;
> > >> return DecodeDPRRegisterClass(Inst, RegNo, Address, Decoder);
> > >> }
> > >>
> > >> -static bool DecodeDPR_VFP2RegisterClass(llvm::MCInst &Inst, unsigned
> > RegNo,
> > >> +static DecodeStatus DecodeDPR_VFP2RegisterClass(llvm::MCInst &Inst,
> > unsigned RegNo,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> if (RegNo > 15)
> > >> - return false;
> > >> + return Fail;
> > >> return DecodeDPRRegisterClass(Inst, RegNo, Address, Decoder);
> > >> }
> > >>
> > >> @@ -677,65 +688,66 @@
> > >> };
> > >>
> > >>
> > >> -static bool DecodeQPRRegisterClass(llvm::MCInst &Inst, unsigned
> RegNo,
> > >> +static DecodeStatus DecodeQPRRegisterClass(llvm::MCInst &Inst,
> unsigned
> > RegNo,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> if (RegNo > 31)
> > >> - return false;
> > >> + return Fail;
> > >> RegNo >>= 1;
> > >>
> > >> unsigned Register = QPRDecoderTable[RegNo];
> > >> Inst.addOperand(MCOperand::CreateReg(Register));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodePredicateOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodePredicateOperand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> - if (Val == 0xF) return false;
> > >> + if (Val == 0xF) return Fail;
> > >> // AL predicate is not allowed on Thumb1 branches.
> > >> if (Inst.getOpcode() == ARM::tBcc && Val == 0xE)
> > >> - return false;
> > >> + return Fail;
> > >> Inst.addOperand(MCOperand::CreateImm(Val));
> > >> if (Val == ARMCC::AL) {
> > >> Inst.addOperand(MCOperand::CreateReg(0));
> > >> } else
> > >> Inst.addOperand(MCOperand::CreateReg(ARM::CPSR));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeCCOutOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeCCOutOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> if (Val)
> > >> Inst.addOperand(MCOperand::CreateReg(ARM::CPSR));
> > >> else
> > >> Inst.addOperand(MCOperand::CreateReg(0));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeSOImmOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeSOImmOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> uint32_t imm = Val & 0xFF;
> > >> uint32_t rot = (Val & 0xF00) >> 7;
> > >> uint32_t rot_imm = (imm >> rot) | (imm << (32-rot));
> > >> Inst.addOperand(MCOperand::CreateImm(rot_imm));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeBLTargetOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeBLTargetOperand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> Val <<= 2;
> > >> Inst.addOperand(MCOperand::CreateImm(SignExtend32<26>(Val)));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeSORegImmOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeSORegImmOperand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >>
> > >> unsigned Rm = fieldFromInstruction32(Val, 0, 4);
> > >> unsigned type = fieldFromInstruction32(Val, 5, 2);
> > >> unsigned imm = fieldFromInstruction32(Val, 7, 5);
> > >>
> > >> // Register-immediate
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >>
> > >> ARM_AM::ShiftOpc Shift = ARM_AM::lsl;
> > >> switch (type) {
> > >> @@ -759,19 +771,20 @@
> > >> unsigned Op = Shift | (imm << 3);
> > >> Inst.addOperand(MCOperand::CreateImm(Op));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeSORegRegOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeSORegRegOperand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >>
> > >> unsigned Rm = fieldFromInstruction32(Val, 0, 4);
> > >> unsigned type = fieldFromInstruction32(Val, 5, 2);
> > >> unsigned Rs = fieldFromInstruction32(Val, 8, 4);
> > >>
> > >> // Register-register
> > >> - if (!DecodeGPRnopcRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> - if (!DecodeGPRnopcRegisterClass(Inst, Rs, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRnopcRegisterClass(Inst, Rm, Address, Decoder));
> > >> + CHECK(S, DecodeGPRnopcRegisterClass(Inst, Rs, Address, Decoder));
> > >>
> > >> ARM_AM::ShiftOpc Shift = ARM_AM::lsl;
> > >> switch (type) {
> > >> @@ -791,49 +804,55 @@
> > >>
> > >> Inst.addOperand(MCOperand::CreateImm(Shift));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeRegListOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeRegListOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> // Empty register lists are not allowed.
> > >> - if (CountPopulation_32(Val) == 0) return false;
> > >> + if (CountPopulation_32(Val) == 0) return Fail;
> > >> for (unsigned i = 0; i < 16; ++i) {
> > >> if (Val & (1 << i)) {
> > >> - if (!DecodeGPRRegisterClass(Inst, i, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, i, Address, Decoder));
> > >> }
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeSPRRegListOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeSPRRegListOperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Vd = fieldFromInstruction32(Val, 8, 4);
> > >> unsigned regs = Val & 0xFF;
> > >>
> > >> - if (!DecodeSPRRegisterClass(Inst, Vd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeSPRRegisterClass(Inst, Vd, Address, Decoder));
> > >> for (unsigned i = 0; i < (regs - 1); ++i) {
> > >> - if (!DecodeSPRRegisterClass(Inst, ++Vd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeSPRRegisterClass(Inst, ++Vd, Address, Decoder));
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeDPRRegListOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeDPRRegListOperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Vd = fieldFromInstruction32(Val, 8, 4);
> > >> unsigned regs = (Val & 0xFF) / 2;
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Vd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Vd, Address, Decoder));
> > >> for (unsigned i = 0; i < (regs - 1); ++i) {
> > >> - if (!DecodeDPRRegisterClass(Inst, ++Vd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, ++Vd, Address, Decoder));
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeBitfieldMaskOperand(llvm::MCInst &Inst, unsigned
> Val,
> > >> +static DecodeStatus DecodeBitfieldMaskOperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> // This operand encodes a mask of contiguous zeros between a
> specified
> > MSB
> > >> // and LSB. To decode it, we create the mask of all bits MSB-and-
> > lower,
> > >> @@ -845,11 +864,13 @@
> > >> uint32_t msb_mask = (1 << (msb+1)) - 1;
> > >> uint32_t lsb_mask = (1 << lsb) - 1;
> > >> Inst.addOperand(MCOperand::CreateImm(~(msb_mask ^ lsb_mask)));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeCopMemInstruction(llvm::MCInst &Inst, unsigned
> Insn,
> > >> +static DecodeStatus DecodeCopMemInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned pred = fieldFromInstruction32(Insn, 28, 4);
> > >> unsigned CRd = fieldFromInstruction32(Insn, 12, 4);
> > >> unsigned coproc = fieldFromInstruction32(Insn, 8, 4);
> > >> @@ -875,7 +896,7 @@
> > >> case ARM::STCL_POST:
> > >> case ARM::STCL_OPTION:
> > >> if (coproc == 0xA || coproc == 0xB)
> > >> - return false;
> > >> + return Fail;
> > >> break;
> > >> default:
> > >> break;
> > >> @@ -883,7 +904,7 @@
> > >>
> > >> Inst.addOperand(MCOperand::CreateImm(coproc));
> > >> Inst.addOperand(MCOperand::CreateImm(CRd));
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> switch (Inst.getOpcode()) {
> > >> case ARM::LDC_OPTION:
> > >> case ARM::LDCL_OPTION:
> > >> @@ -952,17 +973,19 @@
> > >> case ARM::STCL_PRE:
> > >> case ARM::STCL_POST:
> > >> case ARM::STCL_OPTION:
> > >> - if (!DecodePredicateOperand(Inst, pred, Address, Decoder))
> return
> > false;
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeAddrMode2IdxInstruction(llvm::MCInst &Inst,
> unsigned
> > Insn,
> > >> +static DecodeStatus DecodeAddrMode2IdxInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rt = fieldFromInstruction32(Insn, 12, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> @@ -982,13 +1005,13 @@
> > >> case ARM::STRT_POST_IMM:
> > >> case ARM::STRBT_POST_REG:
> > >> case ARM::STRBT_POST_IMM:
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rt, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rt, Address, Decoder));
> > >>
> > >> // On loads, the writeback operand comes after Rt.
> > >> switch (Inst.getOpcode()) {
> > >> @@ -1002,14 +1025,13 @@
> > >> case ARM::LDRBT_POST_IMM:
> > >> case ARM::LDRT_POST_REG:
> > >> case ARM::LDRT_POST_IMM:
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >>
> > >> ARM_AM::AddrOpc Op = ARM_AM::add;
> > >> if (!fieldFromInstruction32(Insn, 23, 1))
> > >> @@ -1022,10 +1044,10 @@
> > >> else if (!P && writeback)
> > >> idx_mode = ARMII::IndexModePost;
> > >>
> > >> - if (writeback && (Rn == 15 || Rn == Rt)) return false; //
> > UNPREDICTABLE
> > >> + if (writeback && (Rn == 15 || Rn == Rt)) S = Unpredictable; //
> > UNPREDICTABLE
> > >>
> > >> if (reg) {
> > >> - if (!DecodeGPRnopcRegisterClass(Inst, Rm, Address, Decoder))
> return
> > false;
> > >> + CHECK(S, DecodeGPRnopcRegisterClass(Inst, Rm, Address, Decoder));
> > >> ARM_AM::ShiftOpc Opc = ARM_AM::lsl;
> > >> switch( fieldFromInstruction32(Insn, 5, 2)) {
> > >> case 0:
> > >> @@ -1041,7 +1063,7 @@
> > >> Opc = ARM_AM::ror;
> > >> break;
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> }
> > >> unsigned amt = fieldFromInstruction32(Insn, 7, 5);
> > >> unsigned imm = ARM_AM::getAM2Opc(Op, amt, Opc, idx_mode);
> > >> @@ -1053,13 +1075,15 @@
> > >> Inst.addOperand(MCOperand::CreateImm(tmp));
> > >> }
> > >>
> > >> - if (!DecodePredicateOperand(Inst, pred, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeSORegMemOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeSORegMemOperand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Val, 13, 4);
> > >> unsigned Rm = fieldFromInstruction32(Val, 0, 4);
> > >> unsigned type = fieldFromInstruction32(Val, 5, 2);
> > >> @@ -1082,8 +1106,8 @@
> > >> break;
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> unsigned shift;
> > >> if (U)
> > >> shift = ARM_AM::getAM2Opc(ARM_AM::add, imm, ShOp);
> > >> @@ -1091,11 +1115,13 @@
> > >> shift = ARM_AM::getAM2Opc(ARM_AM::sub, imm, ShOp);
> > >> Inst.addOperand(MCOperand::CreateImm(shift));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeAddrMode3Instruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> +static DecodeStatus DecodeAddrMode3Instruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rt = fieldFromInstruction32(Insn, 12, 4);
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> @@ -1116,7 +1142,7 @@
> > >> case ARM::LDRD:
> > >> case ARM::LDRD_PRE:
> > >> case ARM::LDRD_POST:
> > >> - if (Rt & 0x1) return false;
> > >> + if (Rt & 0x1) return Fail;
> > >> break;
> > >> default:
> > >> break;
> > >> @@ -1136,16 +1162,14 @@
> > >> case ARM::STRH:
> > >> case ARM::STRH_PRE:
> > >> case ARM::STRH_POST:
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> }
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rt, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rt, Address, Decoder));
> > >> switch (Inst.getOpcode()) {
> > >> case ARM::STRD:
> > >> case ARM::STRD_PRE:
> > >> @@ -1153,8 +1177,7 @@
> > >> case ARM::LDRD:
> > >> case ARM::LDRD_PRE:
> > >> case ARM::LDRD_POST:
> > >> - if (!DecodeGPRRegisterClass(Inst, Rt+1, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rt+1, Address, Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> @@ -1177,33 +1200,32 @@
> > >> case ARM::LDRSB_POST:
> > >> case ARM::LDRHTr:
> > >> case ARM::LDRSBTr:
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> }
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >>
> > >> if (type) {
> > >> Inst.addOperand(MCOperand::CreateReg(0));
> > >> Inst.addOperand(MCOperand::CreateImm(U | (imm << 4) | Rm));
> > >> } else {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(U));
> > >> }
> > >>
> > >> - if (!DecodePredicateOperand(Inst, pred, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeRFEInstruction(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeRFEInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned mode = fieldFromInstruction32(Insn, 23, 2);
> > >>
> > >> @@ -1223,14 +1245,16 @@
> > >> }
> > >>
> > >> Inst.addOperand(MCOperand::CreateImm(mode));
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeMemMultipleWritebackInstruction(llvm::MCInst &Inst,
> > >> +static DecodeStatus
> DecodeMemMultipleWritebackInstruction(llvm::MCInst
> > &Inst,
> > >> unsigned Insn,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned pred = fieldFromInstruction32(Insn, 28, 4);
> > >> unsigned reglist = fieldFromInstruction32(Insn, 0, 16);
> > >> @@ -1265,16 +1289,15 @@
> > >> return DecodeRFEInstruction(Inst, Insn, Address, Decoder);
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder) ||
> > >> - !DecodeGPRRegisterClass(Inst, Rn, Address, Decoder) || //
> Tied
> > >> - !DecodePredicateOperand(Inst, pred, Address, Decoder) ||
> > >> - !DecodeRegListOperand(Inst, reglist, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)); //
> Tied
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >> + CHECK(S, DecodeRegListOperand(Inst, reglist, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeCPSInstruction(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeCPSInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> unsigned imod = fieldFromInstruction32(Insn, 18, 2);
> > >> unsigned M = fieldFromInstruction32(Insn, 17, 1);
> > >> @@ -1282,30 +1305,32 @@
> > >> unsigned mode = fieldFromInstruction32(Insn, 0, 5);
> > >>
> > >> // imod == '01' --> UNPREDICTABLE
> > >> - if (imod == 1) return false;
> > >> + if (imod == 1) return Fail;
> > >>
> > >> if (M && mode && imod && iflags) {
> > >> Inst.setOpcode(ARM::CPS3p);
> > >> Inst.addOperand(MCOperand::CreateImm(imod));
> > >> Inst.addOperand(MCOperand::CreateImm(iflags));
> > >> Inst.addOperand(MCOperand::CreateImm(mode));
> > >> - return true;
> > >> + return Success;
> > >> } else if (!mode && !M) {
> > >> Inst.setOpcode(ARM::CPS2p);
> > >> Inst.addOperand(MCOperand::CreateImm(imod));
> > >> Inst.addOperand(MCOperand::CreateImm(iflags));
> > >> - return true;
> > >> + return Success;
> > >> } else if (!imod && !iflags && M) {
> > >> Inst.setOpcode(ARM::CPS1p);
> > >> Inst.addOperand(MCOperand::CreateImm(mode));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> - return false;
> > >> + return Fail;
> > >> }
> > >>
> > >> -static bool DecodeSMLAInstruction(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeSMLAInstruction(llvm::MCInst &Inst,
> unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rn = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 8, 4);
> > >> @@ -1315,57 +1340,60 @@
> > >> if (pred == 0xF)
> > >> return DecodeCPSInstruction(Inst, Insn, Address, Decoder);
> > >>
> > >> - if (!DecodeGPRnopcRegisterClass(Inst, Rd, Address, Decoder) ||
> > >> - !DecodeGPRnopcRegisterClass(Inst, Rn, Address, Decoder) ||
> > >> - !DecodeGPRnopcRegisterClass(Inst, Rm, Address, Decoder) ||
> > >> - !DecodeGPRnopcRegisterClass(Inst, Ra, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRnopcRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeGPRnopcRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecodeGPRnopcRegisterClass(Inst, Rm, Address, Decoder));
> > >> + CHECK(S, DecodeGPRnopcRegisterClass(Inst, Ra, Address, Decoder));
> > >>
> > >> - if (!DecodePredicateOperand(Inst, pred, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeAddrModeImm12Operand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> +static DecodeStatus DecodeAddrModeImm12Operand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned add = fieldFromInstruction32(Val, 12, 1);
> > >> unsigned imm = fieldFromInstruction32(Val, 0, 12);
> > >> unsigned Rn = fieldFromInstruction32(Val, 13, 4);
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >>
> > >> if (!add) imm *= -1;
> > >> if (imm == 0 && !add) imm = INT32_MIN;
> > >> Inst.addOperand(MCOperand::CreateImm(imm));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeAddrMode5Operand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeAddrMode5Operand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Val, 9, 4);
> > >> unsigned U = fieldFromInstruction32(Val, 8, 1);
> > >> unsigned imm = fieldFromInstruction32(Val, 0, 8);
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >>
> > >> if (U)
> > >> Inst.addOperand(MCOperand::CreateImm(ARM_AM::getAM5Opc(ARM_AM::add,
> > imm)));
> > >> else
> > >> Inst.addOperand(MCOperand::CreateImm(ARM_AM::getAM5Opc(ARM_AM::sub,
> > imm)));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeAddrMode7Operand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeAddrMode7Operand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> return DecodeGPRRegisterClass(Inst, Val, Address, Decoder);
> > >> }
> > >>
> > >> -static bool DecodeBranchImmInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> +static DecodeStatus DecodeBranchImmInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned pred = fieldFromInstruction32(Insn, 28, 4);
> > >> unsigned imm = fieldFromInstruction32(Insn, 0, 24) << 2;
> > >>
> > >> @@ -1373,39 +1401,42 @@
> > >> Inst.setOpcode(ARM::BLXi);
> > >> imm |= fieldFromInstruction32(Insn, 24, 1) << 1;
> > >> Inst.addOperand(MCOperand::CreateImm(SignExtend32<26>(imm)));
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> Inst.addOperand(MCOperand::CreateImm(SignExtend32<26>(imm)));
> > >> - if (!DecodePredicateOperand(Inst, pred, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >>
> > >> -static bool DecodeVCVTImmOperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeVCVTImmOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> Inst.addOperand(MCOperand::CreateImm(64 - Val));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeAddrMode6Operand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeAddrMode6Operand(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rm = fieldFromInstruction32(Val, 0, 4);
> > >> unsigned align = fieldFromInstruction32(Val, 4, 2);
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> if (!align)
> > >> Inst.addOperand(MCOperand::CreateImm(0));
> > >> else
> > >> Inst.addOperand(MCOperand::CreateImm(4 << align));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVLDInstruction(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVLDInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> Rd |= fieldFromInstruction32(Insn, 22, 1) << 4;
> > >> unsigned wb = fieldFromInstruction32(Insn, 16, 4);
> > >> @@ -1414,7 +1445,7 @@
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >>
> > >> // First output register
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >>
> > >> // Second output register
> > >> switch (Inst.getOpcode()) {
> > >> @@ -1466,7 +1497,7 @@
> > >> case ARM::VLD4d8_UPD:
> > >> case ARM::VLD4d16_UPD:
> > >> case ARM::VLD4d32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+1)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+1)%32, Address,
> > Decoder));
> > >> break;
> > >> case ARM::VLD2b8:
> > >> case ARM::VLD2b16:
> > >> @@ -1486,7 +1517,7 @@
> > >> case ARM::VLD4q8_UPD:
> > >> case ARM::VLD4q16_UPD:
> > >> case ARM::VLD4q32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+2)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+2)%32, Address,
> > Decoder));
> > >> default:
> > >> break;
> > >> }
> > >> @@ -1527,7 +1558,7 @@
> > >> case ARM::VLD4d8_UPD:
> > >> case ARM::VLD4d16_UPD:
> > >> case ARM::VLD4d32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+2)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+2)%32, Address,
> > Decoder));
> > >> break;
> > >> case ARM::VLD3q8:
> > >> case ARM::VLD3q16:
> > >> @@ -1541,7 +1572,7 @@
> > >> case ARM::VLD4q8_UPD:
> > >> case ARM::VLD4q16_UPD:
> > >> case ARM::VLD4q32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+4)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+4)%32, Address,
> > Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> @@ -1569,7 +1600,7 @@
> > >> case ARM::VLD4d8_UPD:
> > >> case ARM::VLD4d16_UPD:
> > >> case ARM::VLD4d32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+3)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+3)%32, Address,
> > Decoder));
> > >> break;
> > >> case ARM::VLD4q8:
> > >> case ARM::VLD4q16:
> > >> @@ -1577,7 +1608,7 @@
> > >> case ARM::VLD4q8_UPD:
> > >> case ARM::VLD4q16_UPD:
> > >> case ARM::VLD4q32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+6)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+6)%32, Address,
> > Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> @@ -1622,28 +1653,29 @@
> > >> case ARM::VLD4q8_UPD:
> > >> case ARM::VLD4q16_UPD:
> > >> case ARM::VLD4q32_UPD:
> > >> - if (!DecodeGPRRegisterClass(Inst, wb, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, wb, Address, Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> }
> > >>
> > >> // AddrMode6 Base (register+alignment)
> > >> - if (!DecodeAddrMode6Operand(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeAddrMode6Operand(Inst, Rn, Address, Decoder));
> > >>
> > >> // AddrMode6 Offset (register)
> > >> if (Rm == 0xD)
> > >> Inst.addOperand(MCOperand::CreateReg(0));
> > >> else if (Rm != 0xF) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVSTInstruction(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVSTInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> Rd |= fieldFromInstruction32(Insn, 22, 1) << 4;
> > >> unsigned wb = fieldFromInstruction32(Insn, 16, 4);
> > >> @@ -1690,25 +1722,24 @@
> > >> case ARM::VST4q8_UPD:
> > >> case ARM::VST4q16_UPD:
> > >> case ARM::VST4q32_UPD:
> > >> - if (!DecodeGPRRegisterClass(Inst, wb, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, wb, Address, Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> }
> > >>
> > >> // AddrMode6 Base (register+alignment)
> > >> - if (!DecodeAddrMode6Operand(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeAddrMode6Operand(Inst, Rn, Address, Decoder));
> > >>
> > >> // AddrMode6 Offset (register)
> > >> if (Rm == 0xD)
> > >> Inst.addOperand(MCOperand::CreateReg(0));
> > >> else if (Rm != 0xF) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> // First input register
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >>
> > >> // Second input register
> > >> switch (Inst.getOpcode()) {
> > >> @@ -1760,7 +1791,7 @@
> > >> case ARM::VST4d8_UPD:
> > >> case ARM::VST4d16_UPD:
> > >> case ARM::VST4d32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+1)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+1)%32, Address,
> > Decoder));
> > >> break;
> > >> case ARM::VST2b8:
> > >> case ARM::VST2b16:
> > >> @@ -1780,7 +1811,7 @@
> > >> case ARM::VST4q8_UPD:
> > >> case ARM::VST4q16_UPD:
> > >> case ARM::VST4q32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+2)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+2)%32, Address,
> > Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> @@ -1822,7 +1853,7 @@
> > >> case ARM::VST4d8_UPD:
> > >> case ARM::VST4d16_UPD:
> > >> case ARM::VST4d32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+2)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+2)%32, Address,
> > Decoder));
> > >> break;
> > >> case ARM::VST3q8:
> > >> case ARM::VST3q16:
> > >> @@ -1836,7 +1867,7 @@
> > >> case ARM::VST4q8_UPD:
> > >> case ARM::VST4q16_UPD:
> > >> case ARM::VST4q32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+4)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+4)%32, Address,
> > Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> @@ -1864,7 +1895,7 @@
> > >> case ARM::VST4d8_UPD:
> > >> case ARM::VST4d16_UPD:
> > >> case ARM::VST4d32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+3)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+3)%32, Address,
> > Decoder));
> > >> break;
> > >> case ARM::VST4q8:
> > >> case ARM::VST4q16:
> > >> @@ -1872,17 +1903,19 @@
> > >> case ARM::VST4q8_UPD:
> > >> case ARM::VST4q16_UPD:
> > >> case ARM::VST4q32_UPD:
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+6)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+6)%32, Address,
> > Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVLD1DupInstruction(llvm::MCInst &Inst, unsigned
> Insn,
> > >> +static DecodeStatus DecodeVLD1DupInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> Rd |= fieldFromInstruction32(Insn, 22, 1) << 4;
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> @@ -1893,28 +1926,30 @@
> > >>
> > >> align *= (1 << size);
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> if (regs == 2) {
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+1)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+1)%32, Address,
> > Decoder));
> > >> }
> > >> if (Rm == 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >>
> > >> if (Rm == 0xD)
> > >> Inst.addOperand(MCOperand::CreateReg(0));
> > >> else if (Rm != 0xF) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVLD2DupInstruction(llvm::MCInst &Inst, unsigned
> Insn,
> > >> +static DecodeStatus DecodeVLD2DupInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> Rd |= fieldFromInstruction32(Insn, 22, 1) << 4;
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> @@ -1924,54 +1959,57 @@
> > >> unsigned inc = fieldFromInstruction32(Insn, 5, 1) + 1;
> > >> align *= 2*size;
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rd+inc)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+inc)%32, Address,
> > Decoder));
> > >> if (Rm == 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >>
> > >> if (Rm == 0xD)
> > >> Inst.addOperand(MCOperand::CreateReg(0));
> > >> else if (Rm != 0xF) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVLD3DupInstruction(llvm::MCInst &Inst, unsigned
> Insn,
> > >> +static DecodeStatus DecodeVLD3DupInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> Rd |= fieldFromInstruction32(Insn, 22, 1) << 4;
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned inc = fieldFromInstruction32(Insn, 5, 1) + 1;
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder) ||
> > >> - !DecodeDPRRegisterClass(Inst, (Rd+inc)%32, Address, Decoder) ||
> > >> - !DecodeDPRRegisterClass(Inst, (Rd+2*inc)%32, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+inc)%32, Address,
> > Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+2*inc)%32, Address,
> > Decoder));
> > >> if (Rm == 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(0));
> > >>
> > >> if (Rm == 0xD)
> > >> Inst.addOperand(MCOperand::CreateReg(0));
> > >> else if (Rm != 0xF) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVLD4DupInstruction(llvm::MCInst &Inst, unsigned
> Insn,
> > >> +static DecodeStatus DecodeVLD4DupInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> Rd |= fieldFromInstruction32(Insn, 22, 1) << 4;
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> @@ -1993,29 +2031,30 @@
> > >> }
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)
> ||
> > >> - !DecodeDPRRegisterClass(Inst, (Rd+inc)%32, Address, Decoder)
> ||
> > >> - !DecodeDPRRegisterClass(Inst, (Rd+2*inc)%32, Address, Decoder)
> ||
> > >> - !DecodeDPRRegisterClass(Inst, (Rd+3*inc)%32, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+inc)%32, Address,
> > Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+2*inc)%32, Address,
> > Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rd+3*inc)%32, Address,
> > Decoder));
> > >> if (Rm == 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >>
> > >> if (Rm == 0xD)
> > >> Inst.addOperand(MCOperand::CreateReg(0));
> > >> else if (Rm != 0xF) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeNEONModImmInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> +static DecodeStatus DecodeNEONModImmInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> Rd |= fieldFromInstruction32(Insn, 22, 1) << 4;
> > >> unsigned imm = fieldFromInstruction32(Insn, 0, 4);
> > >> @@ -2026,9 +2065,9 @@
> > >> unsigned Q = fieldFromInstruction32(Insn, 6, 1);
> > >>
> > >> if (Q) {
> > >> - if (!DecodeQPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeQPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> } else {
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> }
> > >>
> > >> Inst.addOperand(MCOperand::CreateImm(imm));
> > >> @@ -2038,62 +2077,66 @@
> > >> case ARM::VORRiv2i32:
> > >> case ARM::VBICiv4i16:
> > >> case ARM::VBICiv2i32:
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> break;
> > >> case ARM::VORRiv8i16:
> > >> case ARM::VORRiv4i32:
> > >> case ARM::VBICiv8i16:
> > >> case ARM::VBICiv4i32:
> > >> - if (!DecodeQPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeQPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> break;
> > >> default:
> > >> break;
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVSHLMaxInstruction(llvm::MCInst &Inst, unsigned
> Insn,
> > >> +static DecodeStatus DecodeVSHLMaxInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> Rd |= fieldFromInstruction32(Insn, 22, 1) << 4;
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> Rm |= fieldFromInstruction32(Insn, 5, 1) << 4;
> > >> unsigned size = fieldFromInstruction32(Insn, 18, 2);
> > >>
> > >> - if (!DecodeQPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeQPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(8 << size));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeShiftRight8Imm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeShiftRight8Imm(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> Inst.addOperand(MCOperand::CreateImm(8 - Val));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeShiftRight16Imm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeShiftRight16Imm(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> Inst.addOperand(MCOperand::CreateImm(16 - Val));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeShiftRight32Imm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeShiftRight32Imm(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> Inst.addOperand(MCOperand::CreateImm(32 - Val));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeShiftRight64Imm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeShiftRight64Imm(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> Inst.addOperand(MCOperand::CreateImm(64 - Val));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeTBLInstruction(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeTBLInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> Rd |= fieldFromInstruction32(Insn, 22, 1) << 4;
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> @@ -2103,21 +2146,21 @@
> > >> unsigned op = fieldFromInstruction32(Insn, 6, 1);
> > >> unsigned length = fieldFromInstruction32(Insn, 8, 2) + 1;
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> if (op) {
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false; // Writeback
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)); //
> > Writeback
> > >> }
> > >>
> > >> for (unsigned i = 0; i < length; ++i) {
> > >> - if (!DecodeDPRRegisterClass(Inst, (Rn+i)%32, Address, Decoder))
> > return false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, (Rn+i)%32, Address,
> > Decoder));
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rm, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVFPfpImm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeVFPfpImm(llvm::MCInst &Inst, unsigned Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> // The immediate needs to be a fully instantiated float. However,
> the
> > >> // auto-generated decoder is only able to fill in some of the bits
> > >> @@ -2139,102 +2182,110 @@
> > >> fp_conv.integer |= (~b & 0x1) << 30;
> > >>
> > >> Inst.addOperand(MCOperand::CreateFPImm(fp_conv.fp));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeThumbAddSpecialReg(llvm::MCInst &Inst, uint16_t
> Insn,
> > >> +static DecodeStatus DecodeThumbAddSpecialReg(llvm::MCInst &Inst,
> > uint16_t Insn,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned dst = fieldFromInstruction16(Insn, 8, 3);
> > >> unsigned imm = fieldFromInstruction16(Insn, 0, 8);
> > >>
> > >> - if (!DecodetGPRRegisterClass(Inst, dst, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodetGPRRegisterClass(Inst, dst, Address, Decoder));
> > >>
> > >> if (Inst.getOpcode() == ARM::tADR)
> > >> Inst.addOperand(MCOperand::CreateReg(ARM::PC));
> > >> else if (Inst.getOpcode() == ARM::tADDrSPi)
> > >> Inst.addOperand(MCOperand::CreateReg(ARM::SP));
> > >> else
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> Inst.addOperand(MCOperand::CreateImm(imm));
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeThumbBROperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbBROperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> Inst.addOperand(MCOperand::CreateImm(SignExtend32<12>(Val << 1)));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeT2BROperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2BROperand(llvm::MCInst &Inst, unsigned
> Val,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> Inst.addOperand(MCOperand::CreateImm(SignExtend32<21>(Val)));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeThumbCmpBROperand(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbCmpBROperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> Inst.addOperand(MCOperand::CreateImm(SignExtend32<7>(Val << 1)));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeThumbAddrModeRR(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbAddrModeRR(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Val, 0, 3);
> > >> unsigned Rm = fieldFromInstruction32(Val, 3, 3);
> > >>
> > >> - if (!DecodetGPRRegisterClass(Inst, Rn, Address, Decoder) ||
> > >> - !DecodetGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodetGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecodetGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeThumbAddrModeIS(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbAddrModeIS(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Val, 0, 3);
> > >> unsigned imm = fieldFromInstruction32(Val, 3, 5);
> > >>
> > >> - if (!DecodetGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodetGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(imm));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeThumbAddrModePC(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbAddrModePC(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> Inst.addOperand(MCOperand::CreateImm(Val << 2));
> > >>
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeThumbAddrModeSP(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbAddrModeSP(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> Inst.addOperand(MCOperand::CreateReg(ARM::SP));
> > >> Inst.addOperand(MCOperand::CreateImm(Val << 2));
> > >>
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeT2AddrModeSOReg(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2AddrModeSOReg(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Val, 6, 4);
> > >> unsigned Rm = fieldFromInstruction32(Val, 2, 4);
> > >> unsigned imm = fieldFromInstruction32(Val, 0, 2);
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder) ||
> > >> - !DecoderGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecoderGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(imm));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeT2LoadShift(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeT2LoadShift(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> if (Inst.getOpcode() != ARM::t2PLDs) {
> > >> unsigned Rt = fieldFromInstruction32(Insn, 12, 4);
> > >> - if (!DecodeGPRRegisterClass(Inst, Rt, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rt, Address, Decoder));
> > >> }
> > >>
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> @@ -2257,57 +2308,60 @@
> > >> Inst.addOperand(MCOperand::CreateReg(ARM::PC));
> > >> break;
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> }
> > >>
> > >> int imm = fieldFromInstruction32(Insn, 0, 12);
> > >> if (!fieldFromInstruction32(Insn, 23, 1)) imm *= -1;
> > >> Inst.addOperand(MCOperand::CreateImm(imm));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> unsigned addrmode = fieldFromInstruction32(Insn, 4, 2);
> > >> addrmode |= fieldFromInstruction32(Insn, 0, 4) << 2;
> > >> addrmode |= fieldFromInstruction32(Insn, 16, 4) << 6;
> > >> - DecodeT2AddrModeSOReg(Inst, addrmode, Address, Decoder);
> > >> + CHECK(S, DecodeT2AddrModeSOReg(Inst, addrmode, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeT2Imm8S4(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2Imm8S4(llvm::MCInst &Inst, unsigned Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> int imm = Val & 0xFF;
> > >> if (!(Val & 0x100)) imm *= -1;
> > >> Inst.addOperand(MCOperand::CreateImm(imm << 2));
> > >>
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeT2AddrModeImm8s4(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2AddrModeImm8s4(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Val, 9, 4);
> > >> unsigned imm = fieldFromInstruction32(Val, 0, 9);
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder) ||
> > >> - !DecodeT2Imm8S4(Inst, imm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecodeT2Imm8S4(Inst, imm, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeT2Imm8(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2Imm8(llvm::MCInst &Inst, unsigned Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> int imm = Val & 0xFF;
> > >> if (!(Val & 0x100)) imm *= -1;
> > >> Inst.addOperand(MCOperand::CreateImm(imm));
> > >>
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >>
> > >> -static bool DecodeT2AddrModeImm8(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2AddrModeImm8(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Val, 9, 4);
> > >> unsigned imm = fieldFromInstruction32(Val, 0, 9);
> > >>
> > >> @@ -2324,27 +2378,28 @@
> > >> break;
> > >> }
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder) ||
> > >> - !DecodeT2Imm8(Inst, imm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecodeT2Imm8(Inst, imm, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >>
> > >> -static bool DecodeT2AddrModeImm12(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2AddrModeImm12(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Val, 13, 4);
> > >> unsigned imm = fieldFromInstruction32(Val, 0, 12);
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(imm));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >>
> > >> -static bool DecodeThumbAddSPImm(llvm::MCInst &Inst, uint16_t Insn,
> > >> +static DecodeStatus DecodeThumbAddSPImm(llvm::MCInst &Inst, uint16_t
> > Insn,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> unsigned imm = fieldFromInstruction16(Insn, 0, 7);
> > >>
> > >> @@ -2352,30 +2407,32 @@
> > >> Inst.addOperand(MCOperand::CreateReg(ARM::SP));
> > >> Inst.addOperand(MCOperand::CreateImm(imm));
> > >>
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeThumbAddSPReg(llvm::MCInst &Inst, uint16_t Insn,
> > >> +static DecodeStatus DecodeThumbAddSPReg(llvm::MCInst &Inst, uint16_t
> > Insn,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> if (Inst.getOpcode() == ARM::tADDrSP) {
> > >> unsigned Rdm = fieldFromInstruction16(Insn, 0, 3);
> > >> Rdm |= fieldFromInstruction16(Insn, 7, 1) << 3;
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rdm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rdm, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateReg(ARM::SP));
> > >> - if (!DecodeGPRRegisterClass(Inst, Rdm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rdm, Address, Decoder));
> > >> } else if (Inst.getOpcode() == ARM::tADDspr) {
> > >> unsigned Rm = fieldFromInstruction16(Insn, 3, 4);
> > >>
> > >> Inst.addOperand(MCOperand::CreateReg(ARM::SP));
> > >> Inst.addOperand(MCOperand::CreateReg(ARM::SP));
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeThumbCPS(llvm::MCInst &Inst, uint16_t Insn,
> > >> +static DecodeStatus DecodeThumbCPS(llvm::MCInst &Inst, uint16_t Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> unsigned imod = fieldFromInstruction16(Insn, 4, 1) | 0x2;
> > >> unsigned flags = fieldFromInstruction16(Insn, 0, 3);
> > >> @@ -2383,52 +2440,55 @@
> > >> Inst.addOperand(MCOperand::CreateImm(imod));
> > >> Inst.addOperand(MCOperand::CreateImm(flags));
> > >>
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodePostIdxReg(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodePostIdxReg(llvm::MCInst &Inst, unsigned
> Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned add = fieldFromInstruction32(Insn, 4, 1);
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder)) ;
> > >> Inst.addOperand(MCOperand::CreateImm(add));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeThumbBLXOffset(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbBLXOffset(llvm::MCInst &Inst, unsigned
> > Val,
> > >> uint64_t Address, const void *Decoder)
> {
> > >> Inst.addOperand(MCOperand::CreateImm(SignExtend32<22>(Val << 1)));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeCoprocessor(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeCoprocessor(llvm::MCInst &Inst, unsigned
> Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> if (Val == 0xA || Val == 0xB)
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> Inst.addOperand(MCOperand::CreateImm(Val));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeThumbSRImm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeThumbSRImm(llvm::MCInst &Inst, unsigned
> Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> if (Val == 0)
> > >> Inst.addOperand(MCOperand::CreateImm(32));
> > >> else
> > >> Inst.addOperand(MCOperand::CreateImm(Val));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeThumb2BCCInstruction(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> +static DecodeStatus DecodeThumb2BCCInstruction(llvm::MCInst &Inst,
> > unsigned Insn,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned pred = fieldFromInstruction32(Insn, 22, 4);
> > >> if (pred == 0xE || pred == 0xF) {
> > >> unsigned opc = fieldFromInstruction32(Insn, 4, 2);
> > >> switch (opc) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case 0:
> > >> Inst.setOpcode(ARM::t2DSB);
> > >> break;
> > >> @@ -2437,7 +2497,7 @@
> > >> break;
> > >> case 2:
> > >> Inst.setOpcode(ARM::t2ISB);
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> unsigned imm = fieldFromInstruction32(Insn, 0, 4);
> > >> @@ -2450,17 +2510,16 @@
> > >> brtarget |= fieldFromInstruction32(Insn, 16, 6) << 12;
> > >> brtarget |= fieldFromInstruction32(Insn, 26, 1) << 20;
> > >>
> > >> - if (!DecodeT2BROperand(Inst, brtarget, Address, Decoder) ||
> > >> - !DecodePredicateOperand(Inst, pred, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeT2BROperand(Inst, brtarget, Address, Decoder));
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> // Decode a shifted immediate operand. These basically consist
> > >> // of an 8-bit value, and a 4-bit directive that specifies either
> > >> // a splat operation or a rotation.
> > >> -static bool DecodeT2SOImm(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeT2SOImm(llvm::MCInst &Inst, unsigned Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> unsigned ctrl = fieldFromInstruction32(Val, 10, 2);
> > >> if (ctrl == 0) {
> > >> @@ -2488,26 +2547,26 @@
> > >> Inst.addOperand(MCOperand::CreateImm(imm));
> > >> }
> > >>
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeThumbBCCTargetOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> +static DecodeStatus DecodeThumbBCCTargetOperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void
> > *Decoder){
> > >> Inst.addOperand(MCOperand::CreateImm(Val << 1));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeThumbBLTargetOperand(llvm::MCInst &Inst, unsigned
> > Val,
> > >> +static DecodeStatus DecodeThumbBLTargetOperand(llvm::MCInst &Inst,
> > unsigned Val,
> > >> uint64_t Address, const void
> > *Decoder){
> > >> Inst.addOperand(MCOperand::CreateImm(SignExtend32<22>(Val << 1)));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeMemBarrierOption(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeMemBarrierOption(llvm::MCInst &Inst,
> unsigned
> > Val,
> > >> uint64_t Address, const void
> *Decoder)
> > {
> > >> switch (Val) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case 0xF: // SY
> > >> case 0xE: // ST
> > >> case 0xB: // ISH
> > >> @@ -2520,55 +2579,61 @@
> > >> }
> > >>
> > >> Inst.addOperand(MCOperand::CreateImm(Val));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeMSRMask(llvm::MCInst &Inst, unsigned Val,
> > >> +static DecodeStatus DecodeMSRMask(llvm::MCInst &Inst, unsigned Val,
> > >> uint64_t Address, const void *Decoder) {
> > >> - if (!Val) return false;
> > >> + if (!Val) return Fail;
> > >> Inst.addOperand(MCOperand::CreateImm(Val));
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> -static bool DecodeDoubleRegLoad(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeDoubleRegLoad(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rt = fieldFromInstruction32(Insn, 12, 4);
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned pred = fieldFromInstruction32(Insn, 28, 4);
> > >>
> > >> - if ((Rt & 1) || Rt == 0xE || Rn == 0xF) return false;
> > >> + if ((Rt & 1) || Rt == 0xE || Rn == 0xF) return Fail;
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rt, Address, Decoder)) return
> > false;
> > >> - if (!DecodeGPRRegisterClass(Inst, Rt+1, Address, Decoder)) return
> > false;
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> - if (!DecodePredicateOperand(Inst, pred, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rt, Address, Decoder));
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rt+1, Address, Decoder));
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >>
> > >> -static bool DecodeDoubleRegStore(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeDoubleRegStore(llvm::MCInst &Inst, unsigned
> > Insn,
> > >> uint64_t Address, const void
> > *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> unsigned Rt = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned pred = fieldFromInstruction32(Insn, 28, 4);
> > >>
> > >> - if (!DecoderGPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecoderGPRRegisterClass(Inst, Rd, Address, Decoder));
> > >>
> > >> - if ((Rt & 1) || Rt == 0xE || Rn == 0xF) return false;
> > >> - if (Rd == Rn || Rd == Rt || Rd == Rt+1) return false;
> > >> + if ((Rt & 1) || Rt == 0xE || Rn == 0xF) return Fail;
> > >> + if (Rd == Rn || Rd == Rt || Rd == Rt+1) return Fail;
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rt, Address, Decoder)) return
> > false;
> > >> - if (!DecodeGPRRegisterClass(Inst, Rt+1, Address, Decoder)) return
> > false;
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> - if (!DecodePredicateOperand(Inst, pred, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rt, Address, Decoder));
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rt+1, Address, Decoder));
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeSTRPreImm(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeSTRPreImm(llvm::MCInst &Inst, unsigned
> Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rt = fieldFromInstruction32(Insn, 12, 4);
> > >> unsigned imm = fieldFromInstruction32(Insn, 0, 12);
> > >> @@ -2576,18 +2641,20 @@
> > >> imm |= fieldFromInstruction32(Insn, 23, 1) << 12;
> > >> unsigned pred = fieldFromInstruction32(Insn, 28, 4);
> > >>
> > >> - if (Rn == 0xF || Rn == Rt) return false; // UNPREDICTABLE
> > >> + if (Rn == 0xF || Rn == Rt) return Unpredictable; // UNPREDICTABLE
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> - if (!DecodeGPRRegisterClass(Inst, Rt, Address, Decoder)) return
> > false;
> > >> - if (!DecodeAddrModeImm12Operand(Inst, imm, Address, Decoder))
> return
> > false;
> > >> - if (!DecodePredicateOperand(Inst, pred, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rt, Address, Decoder));
> > >> + CHECK(S, DecodeAddrModeImm12Operand(Inst, imm, Address, Decoder));
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeSTRPreReg(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeSTRPreReg(llvm::MCInst &Inst, unsigned
> Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rt = fieldFromInstruction32(Insn, 12, 4);
> > >> unsigned imm = fieldFromInstruction32(Insn, 0, 12);
> > >> @@ -2595,18 +2662,20 @@
> > >> imm |= fieldFromInstruction32(Insn, 23, 1) << 12;
> > >> unsigned pred = fieldFromInstruction32(Insn, 28, 4);
> > >>
> > >> - if (Rn == 0xF || Rn == Rt) return false; // UNPREDICTABLE
> > >> + if (Rn == 0xF || Rn == Rt) return Unpredictable; // UNPREDICTABLE
> > >>
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> - if (!DecodeGPRRegisterClass(Inst, Rt, Address, Decoder)) return
> > false;
> > >> - if (!DecodeSORegMemOperand(Inst, imm, Address, Decoder)) return
> > false;
> > >> - if (!DecodePredicateOperand(Inst, pred, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rt, Address, Decoder));
> > >> + CHECK(S, DecodeSORegMemOperand(Inst, imm, Address, Decoder));
> > >> + CHECK(S, DecodePredicateOperand(Inst, pred, Address, Decoder));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVLD1LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVLD1LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> @@ -2617,47 +2686,47 @@
> > >> unsigned index = 0;
> > >> switch (size) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case 0:
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 5, 3);
> > >> break;
> > >> case 1:
> > >> if (fieldFromInstruction32(Insn, 5, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 6, 2);
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> align = 2;
> > >> break;
> > >> case 2:
> > >> if (fieldFromInstruction32(Insn, 6, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 7, 1);
> > >> if (fieldFromInstruction32(Insn, 4, 2) != 0)
> > >> align = 4;
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> if (Rm != 0xF) { // Writeback
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >> if (Rm != 0xF && Rm != 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(index));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVST1LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVST1LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> @@ -2668,47 +2737,47 @@
> > >> unsigned index = 0;
> > >> switch (size) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case 0:
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 5, 3);
> > >> break;
> > >> case 1:
> > >> if (fieldFromInstruction32(Insn, 5, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 6, 2);
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> align = 2;
> > >> break;
> > >> case 2:
> > >> if (fieldFromInstruction32(Insn, 6, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 7, 1);
> > >> if (fieldFromInstruction32(Insn, 4, 2) != 0)
> > >> align = 4;
> > >> }
> > >>
> > >> if (Rm != 0xF) { // Writeback
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >> if (Rm != 0xF && Rm != 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(index));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >>
> > >> -static bool DecodeVLD2LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVLD2LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> @@ -2720,7 +2789,7 @@
> > >> unsigned inc = 1;
> > >> switch (size) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case 0:
> > >> index = fieldFromInstruction32(Insn, 5, 3);
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> @@ -2735,7 +2804,7 @@
> > >> break;
> > >> case 2:
> > >> if (fieldFromInstruction32(Insn, 5, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 7, 1);
> > >> if (fieldFromInstruction32(Insn, 4, 1) != 0)
> > >> align = 8;
> > >> @@ -2744,28 +2813,28 @@
> > >> break;
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder));
> > >> if (Rm != 0xF) { // Writeback
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >> if (Rm != 0xF && Rm != 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(index));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVST2LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVST2LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> @@ -2777,7 +2846,7 @@
> > >> unsigned inc = 1;
> > >> switch (size) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case 0:
> > >> index = fieldFromInstruction32(Insn, 5, 3);
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> @@ -2792,7 +2861,7 @@
> > >> break;
> > >> case 2:
> > >> if (fieldFromInstruction32(Insn, 5, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 7, 1);
> > >> if (fieldFromInstruction32(Insn, 4, 1) != 0)
> > >> align = 8;
> > >> @@ -2802,26 +2871,26 @@
> > >> }
> > >>
> > >> if (Rm != 0xF) { // Writeback
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >> if (Rm != 0xF && Rm != 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(index));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >>
> > >> -static bool DecodeVLD3LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVLD3LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> @@ -2833,53 +2902,53 @@
> > >> unsigned inc = 1;
> > >> switch (size) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case 0:
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 5, 3);
> > >> break;
> > >> case 1:
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 6, 2);
> > >> if (fieldFromInstruction32(Insn, 5, 1))
> > >> inc = 2;
> > >> break;
> > >> case 2:
> > >> if (fieldFromInstruction32(Insn, 4, 2))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 7, 1);
> > >> if (fieldFromInstruction32(Insn, 6, 1))
> > >> inc = 2;
> > >> break;
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder))
> return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder));
> > >>
> > >> if (Rm != 0xF) { // Writeback
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >> if (Rm != 0xF && Rm != 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder))
> return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(index));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVST3LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVST3LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> @@ -2891,22 +2960,22 @@
> > >> unsigned inc = 1;
> > >> switch (size) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case 0:
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 5, 3);
> > >> break;
> > >> case 1:
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 6, 2);
> > >> if (fieldFromInstruction32(Insn, 5, 1))
> > >> inc = 2;
> > >> break;
> > >> case 2:
> > >> if (fieldFromInstruction32(Insn, 4, 2))
> > >> - return false; // UNDEFINED
> > >> + return Fail; // UNDEFINED
> > >> index = fieldFromInstruction32(Insn, 7, 1);
> > >> if (fieldFromInstruction32(Insn, 6, 1))
> > >> inc = 2;
> > >> @@ -2914,27 +2983,27 @@
> > >> }
> > >>
> > >> if (Rm != 0xF) { // Writeback
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >> if (Rm != 0xF && Rm != 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder))
> return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(index));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >>
> > >> -static bool DecodeVLD4LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVLD4LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> @@ -2946,7 +3015,7 @@
> > >> unsigned inc = 1;
> > >> switch (size) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case 0:
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> align = 4;
> > >> @@ -2968,33 +3037,33 @@
> > >> break;
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder))
> return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+3*inc, Address, Decoder))
> return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+3*inc, Address, Decoder));
> > >>
> > >> if (Rm != 0xF) { // Writeback
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >> if (Rm != 0xF && Rm != 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder))
> return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+3*inc, Address, Decoder))
> return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+3*inc, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(index));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >> -static bool DecodeVST4LN(llvm::MCInst &Inst, unsigned Insn,
> > >> +static DecodeStatus DecodeVST4LN(llvm::MCInst &Inst, unsigned Insn,
> > >> uint64_t Address, const void *Decoder) {
> > >> + DecodeStatus S = Success;
> > >> +
> > >> unsigned Rn = fieldFromInstruction32(Insn, 16, 4);
> > >> unsigned Rm = fieldFromInstruction32(Insn, 0, 4);
> > >> unsigned Rd = fieldFromInstruction32(Insn, 12, 4);
> > >> @@ -3006,7 +3075,7 @@
> > >> unsigned inc = 1;
> > >> switch (size) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case 0:
> > >> if (fieldFromInstruction32(Insn, 4, 1))
> > >> align = 4;
> > >> @@ -3029,22 +3098,20 @@
> > >> }
> > >>
> > >> if (Rm != 0xF) { // Writeback
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> }
> > >> - if (!DecodeGPRRegisterClass(Inst, Rn, Address, Decoder)) return
> > false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rn, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(align));
> > >> if (Rm != 0xF && Rm != 0xD) {
> > >> - if (!DecodeGPRRegisterClass(Inst, Rm, Address, Decoder))
> > >> - return false;
> > >> + CHECK(S, DecodeGPRRegisterClass(Inst, Rm, Address, Decoder));
> > >> }
> > >>
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder)) return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder))
> return
> > false;
> > >> - if (!DecodeDPRRegisterClass(Inst, Rd+3*inc, Address, Decoder))
> return
> > false;
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+inc, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+2*inc, Address, Decoder));
> > >> + CHECK(S, DecodeDPRRegisterClass(Inst, Rd+3*inc, Address, Decoder));
> > >> Inst.addOperand(MCOperand::CreateImm(index));
> > >>
> > >> - return true;
> > >> + return S;
> > >> }
> > >>
> > >>
> > >> Modified: llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.h
> > >> URL: http://llvm.org/viewvc/llvm-
> >
> project/llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.h?rev=137830
> > &r1=137829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.h
> (original)
> > >> +++ llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.h Wed Aug
> 17
> > 12:44:15 2011
> > >> @@ -40,11 +40,11 @@
> > >> }
> > >>
> > >> /// getInstruction - See MCDisassembler.
> > >> - bool getInstruction(MCInst &instr,
> > >> - uint64_t &size,
> > >> - const MemoryObject ®ion,
> > >> - uint64_t address,
> > >> - raw_ostream &vStream) const;
> > >> + DecodeStatus getInstruction(MCInst &instr,
> > >> + uint64_t &size,
> > >> + const MemoryObject ®ion,
> > >> + uint64_t address,
> > >> + raw_ostream &vStream) const;
> > >>
> > >> /// getEDInfo - See MCDisassembler.
> > >> EDInstInfo *getEDInfo() const;
> > >> @@ -64,11 +64,11 @@
> > >> }
> > >>
> > >> /// getInstruction - See MCDisassembler.
> > >> - bool getInstruction(MCInst &instr,
> > >> - uint64_t &size,
> > >> - const MemoryObject ®ion,
> > >> - uint64_t address,
> > >> - raw_ostream &vStream) const;
> > >> + DecodeStatus getInstruction(MCInst &instr,
> > >> + uint64_t &size,
> > >> + const MemoryObject ®ion,
> > >> + uint64_t address,
> > >> + raw_ostream &vStream) const;
> > >>
> > >> /// getEDInfo - See MCDisassembler.
> > >> EDInstInfo *getEDInfo() const;
> > >>
> > >> Modified:
> > llvm/trunk/lib/Target/MBlaze/Disassembler/MBlazeDisassembler.cpp
> > >> URL: http://llvm.org/viewvc/llvm-
> >
> project/llvm/trunk/lib/Target/MBlaze/Disassembler/MBlazeDisassembler.cpp?re
> > v=137830&r1=137829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/lib/Target/MBlaze/Disassembler/MBlazeDisassembler.cpp
> > (original)
> > >> +++ llvm/trunk/lib/Target/MBlaze/Disassembler/MBlazeDisassembler.cpp
> Wed
> > Aug 17 12:44:15 2011
> > >> @@ -493,7 +493,7 @@
> > >> // Public interface for the disassembler
> > >> //
> > >>
> > >> -bool MBlazeDisassembler::getInstruction(MCInst &instr,
> > >> +MCDisassembler::DecodeStatus
> MBlazeDisassembler::getInstruction(MCInst
> > &instr,
> > >> uint64_t &size,
> > >> const MemoryObject ®ion,
> > >> uint64_t address,
> > >> @@ -508,7 +508,7 @@
> > >>
> > >> // We want to read exactly 4 bytes of data.
> > >> if (region.readBytes(address, 4, (uint8_t*)bytes, &read) == -1 ||
> read
> > < 4)
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> // Encoded as a big-endian 32-bit word in the stream.
> > >> insn = (bytes[0]<<24) | (bytes[1]<<16) | (bytes[2]<< 8) |
> > (bytes[3]<<0);
> > >> @@ -517,7 +517,7 @@
> > >> // that it is a valid instruction.
> > >> unsigned opcode = getOPCODE(insn);
> > >> if (opcode == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> instr.setOpcode(opcode);
> > >>
> > >> @@ -529,11 +529,11 @@
> > >> uint64_t tsFlags = MBlazeInsts[opcode].TSFlags;
> > >> switch ((tsFlags & MBlazeII::FormMask)) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >>
> > >> case MBlazeII::FRRRR:
> > >> if (RD == UNSUPPORTED || RA == UNSUPPORTED || RB == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateReg(RB));
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> @@ -541,7 +541,7 @@
> > >>
> > >> case MBlazeII::FRRR:
> > >> if (RD == UNSUPPORTED || RA == UNSUPPORTED || RB == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> instr.addOperand(MCOperand::CreateReg(RB));
> > >> @@ -550,23 +550,23 @@
> > >> case MBlazeII::FRI:
> > >> switch (opcode) {
> > >> default:
> > >> - return false;
> > >> + return Fail;
> > >> case MBlaze::MFS:
> > >> if (RD == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateImm(insn&0x3FFF));
> > >> break;
> > >> case MBlaze::MTS:
> > >> if (RA == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateImm(insn&0x3FFF));
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> break;
> > >> case MBlaze::MSRSET:
> > >> case MBlaze::MSRCLR:
> > >> if (RD == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateImm(insn&0x7FFF));
> > >> break;
> > >> @@ -575,7 +575,7 @@
> > >>
> > >> case MBlazeII::FRRI:
> > >> if (RD == UNSUPPORTED || RA == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> switch (opcode) {
> > >> @@ -592,35 +592,35 @@
> > >>
> > >> case MBlazeII::FCRR:
> > >> if (RA == UNSUPPORTED || RB == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> instr.addOperand(MCOperand::CreateReg(RB));
> > >> break;
> > >>
> > >> case MBlazeII::FCRI:
> > >> if (RA == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> instr.addOperand(MCOperand::CreateImm(getIMM(insn)));
> > >> break;
> > >>
> > >> case MBlazeII::FRCR:
> > >> if (RD == UNSUPPORTED || RB == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateReg(RB));
> > >> break;
> > >>
> > >> case MBlazeII::FRCI:
> > >> if (RD == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateImm(getIMM(insn)));
> > >> break;
> > >>
> > >> case MBlazeII::FCCR:
> > >> if (RB == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RB));
> > >> break;
> > >>
> > >> @@ -630,7 +630,7 @@
> > >>
> > >> case MBlazeII::FRRCI:
> > >> if (RD == UNSUPPORTED || RA == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> instr.addOperand(MCOperand::CreateImm(getSHT(insn)));
> > >> @@ -638,35 +638,35 @@
> > >>
> > >> case MBlazeII::FRRC:
> > >> if (RD == UNSUPPORTED || RA == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> break;
> > >>
> > >> case MBlazeII::FRCX:
> > >> if (RD == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateImm(getFSL(insn)));
> > >> break;
> > >>
> > >> case MBlazeII::FRCS:
> > >> if (RD == UNSUPPORTED || RS == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateReg(RS));
> > >> break;
> > >>
> > >> case MBlazeII::FCRCS:
> > >> if (RS == UNSUPPORTED || RA == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RS));
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> break;
> > >>
> > >> case MBlazeII::FCRCX:
> > >> if (RA == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> instr.addOperand(MCOperand::CreateImm(getFSL(insn)));
> > >> break;
> > >> @@ -677,13 +677,13 @@
> > >>
> > >> case MBlazeII::FCR:
> > >> if (RB == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RB));
> > >> break;
> > >>
> > >> case MBlazeII::FRIR:
> > >> if (RD == UNSUPPORTED || RA == UNSUPPORTED)
> > >> - return false;
> > >> + return Fail;
> > >> instr.addOperand(MCOperand::CreateReg(RD));
> > >> instr.addOperand(MCOperand::CreateImm(getIMM(insn)));
> > >> instr.addOperand(MCOperand::CreateReg(RA));
> > >> @@ -693,7 +693,7 @@
> > >> // We always consume 4 bytes of data on success
> > >> size = 4;
> > >>
> > >> - return true;
> > >> + return Success;
> > >> }
> > >>
> > >> static MCDisassembler *createMBlazeDisassembler(const Target &T) {
> > >>
> > >> Modified:
> llvm/trunk/lib/Target/MBlaze/Disassembler/MBlazeDisassembler.h
> > >> URL: http://llvm.org/viewvc/llvm-
> >
> project/llvm/trunk/lib/Target/MBlaze/Disassembler/MBlazeDisassembler.h?rev=
> > 137830&r1=137829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/lib/Target/MBlaze/Disassembler/MBlazeDisassembler.h
> > (original)
> > >> +++ llvm/trunk/lib/Target/MBlaze/Disassembler/MBlazeDisassembler.h Wed
> > Aug 17 12:44:15 2011
> > >> @@ -40,7 +40,7 @@
> > >> }
> > >>
> > >> /// getInstruction - See MCDisassembler.
> > >> - bool getInstruction(MCInst &instr,
> > >> + MCDisassembler::DecodeStatus getInstruction(MCInst &instr,
> > >> uint64_t &size,
> > >> const MemoryObject ®ion,
> > >> uint64_t address,
> > >>
> > >> Modified: llvm/trunk/lib/Target/X86/Disassembler/X86Disassembler.cpp
> > >> URL: http://llvm.org/viewvc/llvm-
> >
> project/llvm/trunk/lib/Target/X86/Disassembler/X86Disassembler.cpp?rev=1378
> > 30&r1=137829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/lib/Target/X86/Disassembler/X86Disassembler.cpp
> > (original)
> > >> +++ llvm/trunk/lib/Target/X86/Disassembler/X86Disassembler.cpp Wed Aug
> > 17 12:44:15 2011
> > >> @@ -106,11 +106,12 @@
> > >> // Public interface for the disassembler
> > >> //
> > >>
> > >> -bool X86GenericDisassembler::getInstruction(MCInst &instr,
> > >> - uint64_t &size,
> > >> - const MemoryObject
> ®ion,
> > >> - uint64_t address,
> > >> - raw_ostream &vStream)
> const
> > {
> > >> +MCDisassembler::DecodeStatus
> > >> +X86GenericDisassembler::getInstruction(MCInst &instr,
> > >> + uint64_t &size,
> > >> + const MemoryObject ®ion,
> > >> + uint64_t address,
> > >> + raw_ostream &vStream) const {
> > >> InternalInstruction internalInstr;
> > >>
> > >> int ret = decodeInstruction(&internalInstr,
> > >> @@ -123,11 +124,11 @@
> > >>
> > >> if (ret) {
> > >> size = internalInstr.readerCursor - address;
> > >> - return false;
> > >> + return Fail;
> > >> }
> > >> else {
> > >> size = internalInstr.length;
> > >> - return !translateInstruction(instr, internalInstr);
> > >> + return (!translateInstruction(instr, internalInstr)) ? Success :
> > Fail;
> > >> }
> > >> }
> > >>
> > >>
> > >> Modified: llvm/trunk/lib/Target/X86/Disassembler/X86Disassembler.h
> > >> URL: http://llvm.org/viewvc/llvm-
> >
> project/llvm/trunk/lib/Target/X86/Disassembler/X86Disassembler.h?rev=137830
> > &r1=137829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/lib/Target/X86/Disassembler/X86Disassembler.h
> (original)
> > >> +++ llvm/trunk/lib/Target/X86/Disassembler/X86Disassembler.h Wed Aug
> 17
> > 12:44:15 2011
> > >> @@ -112,11 +112,11 @@
> > >> ~X86GenericDisassembler();
> > >>
> > >> /// getInstruction - See MCDisassembler.
> > >> - bool getInstruction(MCInst &instr,
> > >> - uint64_t &size,
> > >> - const MemoryObject ®ion,
> > >> - uint64_t address,
> > >> - raw_ostream &vStream) const;
> > >> + DecodeStatus getInstruction(MCInst &instr,
> > >> + uint64_t &size,
> > >> + const MemoryObject ®ion,
> > >> + uint64_t address,
> > >> + raw_ostream &vStream) const;
> > >>
> > >> /// getEDInfo - See MCDisassembler.
> > >> EDInstInfo *getEDInfo() const;
> > >>
> > >> Modified: llvm/trunk/test/MC/Disassembler/ARM/invalid-LDRB_POST-
> arm.txt
> > >> URL: http://llvm.org/viewvc/llvm-
> > project/llvm/trunk/test/MC/Disassembler/ARM/invalid-LDRB_POST-
> > arm.txt?rev=137830&r1=137829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/test/MC/Disassembler/ARM/invalid-LDRB_POST-arm.txt
> > (original)
> > >> +++ llvm/trunk/test/MC/Disassembler/ARM/invalid-LDRB_POST-arm.txt Wed
> > Aug 17 12:44:15 2011
> > >> @@ -1,4 +1,4 @@
> > >> -# RUN: llvm-mc --disassemble %s -triple=arm-apple-darwin9 |& grep
> > {invalid instruction encoding}
> > >> +# RUN: llvm-mc --disassemble %s -triple=arm-apple-darwin9 |& grep
> > {potentially undefined instruction encoding}
> > >>
> > >> # Opcode=140 Name=LDRB_POST Format=ARM_FORMAT_LDFRM(6)
> > >> # 31 30 29 28 27 26 25 24 23 22 21 20 19 18 17 16 15 14 13 12 11 10
> 9
> > 8 7 6 5 4 3 2 1 0
> > >>
> > >> Modified: llvm/trunk/tools/llvm-mc/Disassembler.cpp
> > >> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-
> > mc/Disassembler.cpp?rev=137830&r1=137829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/tools/llvm-mc/Disassembler.cpp (original)
> > >> +++ llvm/trunk/tools/llvm-mc/Disassembler.cpp Wed Aug 17 12:44:15 2011
> > >> @@ -65,15 +65,26 @@
> > >> for (Index = 0; Index < Bytes.size(); Index += Size) {
> > >> MCInst Inst;
> > >>
> > >> - if (DisAsm.getInstruction(Inst, Size, memoryObject, Index,
> > >> - /*REMOVE*/ nulls())) {
> > >> - Printer.printInst(&Inst, Out);
> > >> - Out << "\n";
> > >> - } else {
> > >> + MCDisassembler::DecodeStatus S;
> > >> + S = DisAsm.getInstruction(Inst, Size, memoryObject, Index,
> > >> + /*REMOVE*/ nulls());
> > >> + switch (S) {
> > >
> > >
> > > Changing this to a switch statement is good. The C bindings in
> > MCDisasembler.cpp and EDDisassembler.cpp need to be changed, too, though.
> > They're still doing "!Disasm->getInstruction()" type constructs.
> > >
> > >
> > >> + case MCDisassembler::Fail:
> > >> SM.PrintMessage(SMLoc::getFromPointer(Bytes[Index].second),
> > >> "invalid instruction encoding", "warning");
> > >> if (Size == 0)
> > >> Size = 1; // skip illegible bytes
> > >> + break;
> > >> +
> > >> + case MCDisassembler::SoftFail:
> > >> + SM.PrintMessage(SMLoc::getFromPointer(Bytes[Index].second),
> > >> + "potentially undefined instruction encoding",
> > "warning");
> > >> + // Fall through
> > >> +
> > >> + case MCDisassembler::Success:
> > >> + Printer.printInst(&Inst, Out);
> > >> + Out << "\n";
> > >> + break;
> > >> }
> > >> }
> > >>
> > >>
> > >> Modified: llvm/trunk/utils/TableGen/DisassemblerEmitter.cpp
> > >> URL: http://llvm.org/viewvc/llvm-
> >
> project/llvm/trunk/utils/TableGen/DisassemblerEmitter.cpp?rev=137830&r1=137
> > 829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/utils/TableGen/DisassemblerEmitter.cpp (original)
> > >> +++ llvm/trunk/utils/TableGen/DisassemblerEmitter.cpp Wed Aug 17
> > 12:44:15 2011
> > >> @@ -128,5 +128,15 @@
> > >> return;
> > >> }
> > >>
> > >> + // ARM and Thumb have a CHECK() macro to deal with DecodeStatuses.
> > >> + if (Target.getName() == "ARM" ||
> > >> + Target.getName() == "Thumb") {
> > >> + FixedLenDecoderEmitter(Records,
> > >> + "CHECK(S, ", ");",
> > >> + "S", "Fail",
> > >> + "DecodeStatus S =
> > Success;\n(void)S;").run(OS);
> > >> + return;
> > >> + }
> > >> +
> > >> FixedLenDecoderEmitter(Records).run(OS);
> > >> }
> > >>
> > >> Modified: llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.cpp
> > >> URL: http://llvm.org/viewvc/llvm-
> >
> project/llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.cpp?rev=137830&r1=
> > 137829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.cpp (original)
> > >> +++ llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.cpp Wed Aug 17
> > 12:44:15 2011
> > >> @@ -238,19 +238,24 @@
> > >> // Width of instructions
> > >> unsigned BitWidth;
> > >>
> > >> + // Parent emitter
> > >> + const FixedLenDecoderEmitter *Emitter;
> > >> +
> > >> public:
> > >> FilterChooser(const FilterChooser &FC) :
> > >> AllInstructions(FC.AllInstructions), Opcodes(FC.Opcodes),
> > >> Operands(FC.Operands), Filters(FC.Filters),
> > >> FilterBitValues(FC.FilterBitValues), Parent(FC.Parent),
> > >> - BestIndex(FC.BestIndex), BitWidth(FC.BitWidth) { }
> > >> + BestIndex(FC.BestIndex), BitWidth(FC.BitWidth),
> > >> + Emitter(FC.Emitter) { }
> > >>
> > >> FilterChooser(const std::vector<const CodeGenInstruction*> &Insts,
> > >> const std::vector<unsigned> &IDs,
> > >> std::map<unsigned, std::vector<OperandInfo> > &Ops,
> > >> - unsigned BW) :
> > >> + unsigned BW,
> > >> + const FixedLenDecoderEmitter *E) :
> > >> AllInstructions(Insts), Opcodes(IDs), Operands(Ops), Filters(),
> > >> - Parent(NULL), BestIndex(-1), BitWidth(BW) {
> > >> + Parent(NULL), BestIndex(-1), BitWidth(BW), Emitter(E) {
> > >> for (unsigned i = 0; i < BitWidth; ++i)
> > >> FilterBitValues.push_back(BIT_UNFILTERED);
> > >>
> > >> @@ -264,7 +269,8 @@
> > >> FilterChooser &parent) :
> > >> AllInstructions(Insts), Opcodes(IDs), Operands(Ops),
> > >> Filters(), FilterBitValues(ParentFilterBitValues),
> > >> - Parent(&parent), BestIndex(-1), BitWidth(parent.BitWidth) {
> > >> + Parent(&parent), BestIndex(-1), BitWidth(parent.BitWidth),
> > >> + Emitter(parent.Emitter) {
> > >> doFilter();
> > >> }
> > >>
> > >> @@ -563,17 +569,17 @@
> > >> void FilterChooser::emitTop(raw_ostream &o, unsigned Indentation,
> > >> std::string Namespace) {
> > >> o.indent(Indentation) <<
> > >> - "static bool decode" << Namespace << "Instruction" << BitWidth
> > >> + "static MCDisassembler::DecodeStatus decode" << Namespace <<
> > "Instruction" << BitWidth
> > >> << "(MCInst &MI, uint" << BitWidth << "_t insn, uint64_t Address, "
> > >> << "const void *Decoder) {\n";
> > >> - o.indent(Indentation) << " unsigned tmp = 0;\n (void)tmp;\n";
> > >> + o.indent(Indentation) << " unsigned tmp = 0;\n (void)tmp;\n" <<
> > Emitter->Locals << "\n";
> > >>
> > >> ++Indentation; ++Indentation;
> > >> // Emits code to decode the instructions.
> > >> emit(o, Indentation);
> > >>
> > >> o << '\n';
> > >> - o.indent(Indentation) << "return false;\n";
> > >> + o.indent(Indentation) << "return " << Emitter->ReturnFail << ";\n";
> > >> --Indentation; --Indentation;
> > >>
> > >> o.indent(Indentation) << "}\n";
> > >> @@ -744,8 +750,8 @@
> > >> }
> > >>
> > >> if (Decoder != "")
> > >> - o.indent(Indentation) << " if (!" << Decoder
> > >> - << "(MI, tmp, Address, Decoder)) return
> > false;\n";
> > >> + o.indent(Indentation) << " " << Emitter->GuardPrefix << Decoder
> > >> + << "(MI, tmp, Address, Decoder)" <<
> Emitter-
> > >GuardPostfix << "\n";
> > >> else
> > >> o.indent(Indentation) << "
> > MI.addOperand(MCOperand::CreateImm(tmp));\n";
> > >>
> > >> @@ -776,15 +782,15 @@
> > >> I = InsnOperands.begin(), E = InsnOperands.end(); I != E; ++I)
> {
> > >> // If a custom instruction decoder was specified, use that.
> > >> if (I->numFields() == 0 && I->Decoder.size()) {
> > >> - o.indent(Indentation) << " if (!" << I->Decoder
> > >> - << "(MI, insn, Address, Decoder))
> return
> > false;\n";
> > >> + o.indent(Indentation) << " " << Emitter->GuardPrefix << I-
> > >Decoder
> > >> + << "(MI, insn, Address, Decoder)" <<
> > Emitter->GuardPostfix << "\n";
> > >> break;
> > >> }
> > >>
> > >> emitBinaryParser(o, Indentation, *I);
> > >> }
> > >>
> > >> - o.indent(Indentation) << " return true; // " << nameWithID(Opc)
> > >> + o.indent(Indentation) << " return " << Emitter->ReturnOK << ";
> //
> > " << nameWithID(Opc)
> > >> << '\n';
> > >> o.indent(Indentation) << "}\n";
> > >> return true;
> > >> @@ -821,14 +827,14 @@
> > >> I = InsnOperands.begin(), E = InsnOperands.end(); I != E; ++I) {
> > >> // If a custom instruction decoder was specified, use that.
> > >> if (I->numFields() == 0 && I->Decoder.size()) {
> > >> - o.indent(Indentation) << " if (!" << I->Decoder
> > >> - << "(MI, insn, Address, Decoder)) return
> > false;\n";
> > >> + o.indent(Indentation) << " " << Emitter->GuardPrefix << I-
> > >Decoder
> > >> + << "(MI, insn, Address, Decoder)" <<
> > Emitter->GuardPostfix << "\n";
> > >> break;
> > >> }
> > >>
> > >> emitBinaryParser(o, Indentation, *I);
> > >> }
> > >> - o.indent(Indentation) << " return true; // " << nameWithID(Opc)
> > >> + o.indent(Indentation) << " return " << Emitter->ReturnOK << "; //
> "
> > << nameWithID(Opc)
> > >> << '\n';
> > >> o.indent(Indentation) << "}\n";
> > >>
> > >> @@ -1426,7 +1432,7 @@
> > >>
> > >> // Emit the decoder for this namespace+width combination.
> > >> FilterChooser FC(NumberedInstructions, I->second, Operands,
> > >> - 8*I->first.second);
> > >> + 8*I->first.second, this);
> > >> FC.emitTop(o, 0, I->first.first);
> > >> }
> > >>
> > >>
> > >> Modified: llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.h
> > >> URL: http://llvm.org/viewvc/llvm-
> >
> project/llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.h?rev=137830&r1=13
> > 7829&r2=137830&view=diff
> > >>
> >
> ===========================================================================
> > ===
> > >> --- llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.h (original)
> > >> +++ llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.h Wed Aug 17
> > 12:44:15 2011
> > >> @@ -49,9 +49,16 @@
> > >>
> > >> class FixedLenDecoderEmitter : public TableGenBackend {
> > >> public:
> > >> - FixedLenDecoderEmitter(RecordKeeper &R) :
> > >> + FixedLenDecoderEmitter(RecordKeeper &R,
> > >> + std::string GPrefix = "if (",
> > >> + std::string GPostfix = " ==
> > MCDisassembler::Fail) return MCDisassembler::Fail;",
> > >> + std::string ROK =
> > "MCDisassembler::Success",
> > >> + std::string RFail =
> "MCDisassembler::Fail",
> > >> + std::string L = "") :
> > >> Records(R), Target(R),
> > >> - NumberedInstructions(Target.getInstructionsByEnumValue()) {}
> > >> + NumberedInstructions(Target.getInstructionsByEnumValue()),
> > >> + GuardPrefix(GPrefix), GuardPostfix(GPostfix),
> > >> + ReturnOK(ROK), ReturnFail(RFail), Locals(L) {}
> > >>
> > >> // run - Output the code emitter
> > >> void run(raw_ostream &o);
> > >> @@ -62,7 +69,10 @@
> > >> std::vector<const CodeGenInstruction*> NumberedInstructions;
> > >> std::vector<unsigned> Opcodes;
> > >> std::map<unsigned, std::vector<OperandInfo> > Operands;
> > >> -
> > >> +public:
> > >> + std::string GuardPrefix, GuardPostfix;
> > >> + std::string ReturnOK, ReturnFail;
> > >> + std::string Locals;
> > >> };
> > >>
> > >> } // end llvm namespace
> > >>
> > >>
> > >> _______________________________________________
> > >> llvm-commits mailing list
> > >> llvm-commits at cs.uiuc.edu
> > >> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> > >
> >
> >
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fixup-softfail.patch
Type: application/octet-stream
Size: 133114 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110901/d140ace8/attachment.obj>
More information about the llvm-commits
mailing list