<p dir="ltr"> Wouldn't this make line tables longer, as it would take extra bits in the line table state machine to change the discriminator back to zero?</p>
<br><div class="gmail_quote"><div dir="ltr">On Thu., 31 Aug. 2017, 5:46 pm Paul Robinson 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">probinson created this revision.<br>
Herald added a subscriber: JDevlieghere.<br>
<br>
We have line 0 inherit the scope of the previous non-zero line, to limit the size cost of the line 0 records; but that scope might have a discriminator attached, and a discriminator on line 0 is pointless.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D37364" rel="noreferrer" target="_blank">https://reviews.llvm.org/D37364</a><br>
<br>
Files:<br>
  lib/CodeGen/AsmPrinter/DwarfDebug.cpp<br>
  test/CodeGen/Generic/dbg-line-0-no-discriminator.ll<br>
<br>
</blockquote></div>