<div dir="ltr">I found 2 more cases when 32-bit offsets are not enough: <span style="color:rgb(0,0,0);font-family:"Segoe UI","Segoe UI Emoji","Segoe UI Symbol",Lato,"Helvetica Neue",Helvetica,Arial,sans-serif;font-size:13px">DeclOffests and TypeOffssts. They belong to </span>DECLTYPES_BLOCK block that usually takes about 2/3 of the file and on 900M preamble file exceeds 512M alone. In such cases I'm using 64-bit offsets.<div>Diff <a href="https://reviews.llvm.org/D76594" target="_blank">https://reviews.llvm.org/D76594</a> was updated to handle these cases. If you are interested in large AST files support, please review.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 23, 2020 at 9:09 AM Dmitry Polukhin <<a href="mailto:dmitry.polukhin@gmail.com">dmitry.polukhin@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><a href="https://reviews.llvm.org/D76594" target="_blank">https://reviews.llvm.org/D76594</a> implements relative offsets for 32-bit bit offsets. I tried to find all potentially problematic places and found one more that I didn't identify initially <span style="color:rgb(0,0,0);font-family:Menlo,Consolas,Monaco,monospace;font-size:11px;white-space:pre-wrap">PPD_ENTITIES_OFFSETS.</span></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 18, 2020 at 5:42 PM Dmitry Polukhin <<a href="mailto:dmitry.polukhin@gmail.com" target="_blank">dmitry.polukhin@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">OK, it seems that there is kind of consensus that second option is better and additional complexity worths it. My main concern about second option is that it is kind of temporary solution that just delays the point then 32bit won't be enough for bit offsets.<div>I'll explore second option in more details and post results here.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 18, 2020 at 1:13 AM Bruno Cardoso Lopes <<a href="mailto:bruno.cardoso@gmail.com" target="_blank">bruno.cardoso@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><br>On Tue, Mar 17, 2020 at 12:57 PM Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>> wrote:<br>><br>> +Michael, Bruno, and Volodymyr<br>><br>> I agree with Sam that option 2 seems cleaner.<div><br></div><div>+1<br></div><div><br></div><div>> I'm not sure whether 4% (option 1) would be a blocker for us, but probably it would be fine.</div><div><br></div><div>Ultimately I believe it would be fine.</div><div><br>-- <br>Bruno Cardoso Lopes<br><a href="http://www.brunocardoso.cc" target="_blank">http://www.brunocardoso.cc</a></div></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>