[llvm-commits] CVS: llvm/lib/Bytecode/Reader/Reader.h

[Reid Spencer]

Changes in directory llvm/lib/Bytecode/Reader:

Reader.h updated: 1.12 -> 1.13
---
Log message:

Add boolean file format flags in preparation for version 5 bytecode.
Remove the "processFunctions" boolean from ParseBytecode as it is no
longer needed. This is part of avoiding double reading of functions 
when analyzing bytecode.


---
Diffs of the changes:  (+18 -3)

Index: llvm/lib/Bytecode/Reader/Reader.h
diff -u llvm/lib/Bytecode/Reader/Reader.h:1.12 llvm/lib/Bytecode/Reader/Reader.h:1.13
--- llvm/lib/Bytecode/Reader/Reader.h:1.12	Tue Aug 17 02:45:14 2004
+++ llvm/lib/Bytecode/Reader/Reader.h	Sat Aug 21 15:50:49 2004
@@ -130,8 +130,7 @@
   void ParseBytecode(
      const unsigned char *Buf,    ///< Beginning of the bytecode buffer
      unsigned Length,             ///< Length of the bytecode buffer
-     const std::string &ModuleID, ///< An identifier for the module constructed.
-     bool processFunctions=false  ///< Process all function bodies fully.
+     const std::string &ModuleID  ///< An identifier for the module constructed.
   );
 
   /// @brief Parse all function bodies
@@ -299,6 +298,21 @@
   /// alignment of bytecode fields was done away with completely.
   bool hasAlignment;
 
+  // In version 4, basic blocks have a minimum index of 0 whereas all the 
+  // other primitives have a minimum index of 1 (because 0 is the "null" 
+  // value. In version 5, we made this consistent.
+  bool hasInconsistentBBSlotNums;
+
+  // In version 4, the types SByte and UByte were encoded as vbr_uint so that
+  // signed values > 63 and unsigned values >127 would be encoded as two
+  // bytes. In version 5, they are encoded directly in a single byte.
+  bool hasVBRByteTypes;
+
+  // In version 4, modules begin with a "Module Block" which encodes a 4-byte
+  // integer value 0x01 to identify the module block. This is unnecessary and
+  // removed in version 5.
+  bool hasUnnecessaryModuleBlockId;
+
   /// CompactionTypes - If a compaction table is active in the current function,
   /// this is the mapping that it contains.  We keep track of what resolved type
   /// it is as well as what global type entry it is.
@@ -498,7 +512,8 @@
 
 /// @brief A function for creating a BytecodeAnalzer as a handler
 /// for the Bytecode reader.
-BytecodeHandler* createBytecodeAnalyzerHandler(BytecodeAnalysis& bca );
+BytecodeHandler* createBytecodeAnalyzerHandler(BytecodeAnalysis& bca, 
+                                               std::ostream* output );
 
 
 } // End llvm namespace