<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 12, 2014 at 10:30 AM, Adrian Prantl <span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi David,<div><br></div><div>I committed the improved version of the test in r201248.</div>
</div></blockquote><div><br></div><div>Thanks for that - sorry for the disruption/inconvenience.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">
<span class="HOEnZb"><font color="#888888"><div><br></div><div>-- adrian</div></font></span><div><div class="h5"><div><br><div><div>On Feb 12, 2014, at 10:26, Adrian Prantl <<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>> wrote:</div>
<br><blockquote type="cite"><div style="word-wrap:break-word"><br><div><div>On Feb 11, 2014, at 21:12, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:</div><br><blockquote type="cite">
<div dir="ltr">> So my best guess is that this test became useless at some point (possibly at 139468) due to refactoring and that the issue is currently covered by debuginfo-properties.m.<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>
<br>
</div>AFAICT debuginfo-properties.m checks for the DW_AT_decl_line in the debug info, whereas this tests for a line table entry. I forgot to mention that the radar was about being able to set a breakpoint on the property.</blockquote>


<div><br></div><div>OK - though I wasn't able to get the test case to fail. I suspect because something else ended up with the same line information, perhaps? Or it ended up with the correct line info through some other means? (I didn't diff the full output to see if it changed)<br>

<br>The commit the test case was addressing didn't seem like it was doing anything line-table specific... I assume it was just testing the line table because it was a convenient way to test the line associated with this construct (I assume this was originally a grep test long before llvm-dwarfdump for example).</div>


</div></div></div>
</blockquote></div><br><div><div style="margin:0px">I'm rewriting the debug-property-synth.m test to be IR-based (and documented!) now — based on the description in the radar.</div><div style="margin:0px">Passing the StartLoc all the way through affects both the line table location _and_ the DW_AT_decl_line of the function (which is read from the CGDebugInfo::CurLoc).</div>
</div><div style="margin:0px"><div style="margin:0px"><br></div><div style="margin:0px">  void CodeGenFunction::StartFunction(...</div><div style="margin:0px">  ...</div><div style="margin:0px">      DI->setLocation(StartLoc);</div>
<div style="margin:0px">      DI->EmitFunctionStart(GD, FnType, CurFn, Builder);</div><div style="margin:0px">    }</div><div><br></div><div>-- adrian</div></div></div></blockquote></div><br></div></div></div></div></blockquote>
</div><br></div></div>