<div dir="ltr">Anyone thought about whether it'd be worth making the indexed V unindexed case work implicitly? If we were willing to add another pointer to DwarfStringPoolEntryRef (doubling its size, admittedly) - back to the string pool itself - then we could index any string where the DwarfStringPoolEntryRef was asked for an index, and otherwise skip that? Rather than having to declare up-front whether an index was required.<br><br>(if we wanted to get super-fancy, maybe we could emit indexes as assembly expressions ((str_label/str_begin_label)/word_size) then you wouldn't have to increase the size of DwarfStringPoolEntryRef, because the absolute index wouldn't have to be known ahead of time - it'd be computed by the assembler instead - which is just moving the work around, of course)</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 19, 2018 at 9:57 AM Jonas Devlieghere via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">JDevlieghere accepted this revision.<br>
JDevlieghere added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
I'm happy with this, LGTM!<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D49493" rel="noreferrer" target="_blank">https://reviews.llvm.org/D49493</a><br>
<br>
<br>
<br>
</blockquote></div>