<br><br><div class="gmail_quote">On Wed, Dec 24, 2008 at 7:29 AM, Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com">kremenek@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Part of the problem is that "Deserialize.cpp" (in llvm) uses GetCurrentBitNo() in a whole bunch of places.  AST serialization/deserialization is the only client of the serialization library, and there aren't that many test cases in the clang test suite for AST serialization because it just hasn't been completed yet.  I have a feeling that changing the behavior of GetCurrentBitNo() will break things.<br>

<br>
I'm actually a little surprised that GetCurrentBitNo() needs to be changed.  While the AST serialization work is incomplete, it has worked for non-trivial cases.  Since it uses GetCurrentBitNo() to jump around the bitcode file, I'm suspicious that anything needs to be changed.<br>

<br>
Also, what happens if NextChar == FirstChar?  Wouldn't (NextChar-FirstChar-4)*8 be a negative number?</blockquote><div><br>I don't know. Let's leave it as it is now.<br> </div></div><br>