On Wed, Oct 10, 2012 at 9:59 PM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com" target="_blank">clattner@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div style="word-wrap:break-word"><br><div><div><div>On Oct 10, 2012, at 5:10 PM, Jan Voung <<a href="mailto:jvoung@chromium.org" target="_blank">jvoung@chromium.org</a>> wrote:</div><br><blockquote type="cite">
Thanks for the review Rafael!<div><br></div><div>Chris, did you want to take a look at the patch too?</div></blockquote><div><br></div></div><div>Your most recent patch looks good to me, with a minor change:</div><div><br>

</div><div><div style="margin:0px;font-size:11px;font-family:Menlo">+++ b/lib/Bitcode/Writer/BitcodeWriter.cpp</div><div style="margin:0px;font-size:11px;font-family:Menlo">@@ -33,6 +33,13 @@</div><div style="margin:0px;font-size:11px;font-family:Menlo">

 using namespace llvm;</div><p style="margin:0px;font-size:11px;font-family:Menlo;min-height:13px"> <br></p><div style="margin:0px;font-size:11px;font-family:Menlo"> static cl::opt<bool></div><div style="margin:0px;font-size:11px;font-family:Menlo">

+UseAbsoluteOperandIDs("use-abs-operands",</div><div style="margin:0px;font-size:11px;font-family:Menlo">+                      cl::desc("Assign instruction operand IDs with the older "</div><div style="margin:0px;font-size:11px;font-family:Menlo">

+                               "absolute encoding instead of using "</div><div style="margin:0px;font-size:11px;font-family:Menlo">+                               "relative IDs."),</div><div style="margin:0px;font-size:11px;font-family:Menlo">

+                      cl::init(false), cl::Hidden);</div><div><br></div><div>Please remove this, the new writer should just always generate the relative encoding.  Please feel free to land the patch with this change.</div>

<div> <br></div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div><div></div><div>Also, this is a significant enough thing that it would be good to add something to the bitcode docs (in the IR section, <a href="http://llvm.org/docs/BitCodeFormat.html#llvm-ir-encoding" target="_blank">http://llvm.org/docs/BitCodeFormat.html#llvm-ir-encoding</a>) that describes the old and new format.</div>

<div><br></div><div>Thanks again for tackling this!</div><span><font color="#888888"><div><br></div><div>-Chris</div></font></span></div></div></div></blockquote></div><br></div><div class="gmail_extra"><div><span style="font-family:arial,sans-serif;font-size:13px">Thanks!  </span>Removed the flag and committed as <span style="font-family:arial,sans-serif;font-size:13px">165739.  Will update the docs.</span></div>
<div><br></div><div>- Jan</div></div>