Oops, thanks Duncan -- fixed.<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 11, 2012 at 1:32 PM, Duncan Sands <span dir="ltr"><<a href="mailto:baldrick@free.fr" target="_blank">baldrick@free.fr</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jan,<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Change encoding of instruction operands in bitcode binaries to be relative<br>
to the instruction position.  The old encoding would give an absolute<br>
ID which counts up within a function, and only resets at the next function.<br>
</blockquote>
<br></div>
it looks great!  I did notice the following typos:<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
--- llvm/trunk/lib/Bitcode/Reader/<u></u>BitcodeReader.h (original)<br>
+++ llvm/trunk/lib/Bitcode/Reader/<u></u>BitcodeReader.h Thu Oct 11 15:20:40 2012<br>
@@ -179,18 +179,27 @@<br>
    typedef std::pair<unsigned, GlobalVariable*> BlockAddrRefTy;<br>
    DenseMap<Function*, std::vector<BlockAddrRefTy> > BlockAddrFwdRefs;<br>
<br>
+  /// UseRelativeIDs - Indicates that we are using a new encoding for<br>
+  /// instrunction operands where most operands in the current<br>
</blockquote>
<br></div>
instrunction -> instruction<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+  /// FUNCTION_BLOCK are encoded relative to the instruction number,<br>
+  /// for a more compact encoding.  Some instruction operands are not<br>
+  /// relative to the instruction ID: basic block numbers, and types.<br>
+  /// Once the old style function blocks have been phased out, we would<br>
+  /// not need this flag.<br>
+  bool UseRelativeIDs;<br>
</blockquote></div>
...<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
@@ -1514,8 +1551,8 @@<br>
  // Emit blockinfo, which defines the standard abbreviations etc.<br>
  static void WriteBlockInfo(const ValueEnumerator &VE, BitstreamWriter &Stream) {<br>
    // We only want to emit block info records for blocks that have multiple<br>
-  // instances: CONSTANTS_BLOCK, FUNCTION_BLOCK and VALUE_SYMTAB_BLOCK.  Other<br>
-  // blocks can defined their abbrevs inline.<br>
+  // instances: CONSTANTS_BLOCK, FUNCTION_BLOCK and VALUE_SYMTAB_BLOCK.<br>
+  // Other blocks can defined their abbrevs inline.<br>
</blockquote>
<br></div>
can defined -> can define<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    Stream.EnterBlockInfoBlock(2);<br>
<br>
    { // 8-bit fixed-width VST_ENTRY/VST_BBENTRY strings.<br>
</blockquote>
<br></div>
Ciao, Duncan.<div class="HOEnZb"><div class="h5"><br>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>