[llvm] r201039 - Remove some unnecessary code. The conditions it was checking had already been ruled out by the caller.
Craig Topper
craig.topper at gmail.com
Sat Feb 8 23:13:42 PST 2014
Author: ctopper
Date: Sun Feb 9 01:13:41 2014
New Revision: 201039
URL: http://llvm.org/viewvc/llvm-project?rev=201039&view=rev
Log:
Remove some unnecessary code. The conditions it was checking had already been ruled out by the caller.
Modified:
llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.cpp
Modified: llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.cpp?rev=201039&r1=201038&r2=201039&view=diff
==============================================================================
--- llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.cpp (original)
+++ llvm/trunk/utils/TableGen/FixedLenDecoderEmitter.cpp Sun Feb 9 01:13:41 2014
@@ -1705,13 +1705,6 @@ static bool populateInstruction(CodeGenT
// If all the bit positions are not specified; do not decode this instruction.
// We are bound to fail! For proper disassembly, the well-known encoding bits
// of the instruction must be fully specified.
- //
- // This also removes pseudo instructions from considerations of disassembly,
- // which is a better design and less fragile than the name matchings.
- // Ignore "asm parser only" instructions.
- if (Def.getValueAsBit("isAsmParserOnly") ||
- Def.getValueAsBit("isCodeGenOnly"))
- return false;
BitsInit &Bits = getBitsField(Def, "Inst");
if (Bits.allInComplete()) return false;
More information about the llvm-commits
mailing list