[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