<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 10, 2014 at 12:44 PM, Tom Stellard <span dir="ltr"><<a href="mailto:tom@stellard.net" target="_blank">tom@stellard.net</a>></span> wrote:<br><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"><span class="">On Fri, Oct 10, 2014 at 11:08:19AM -0700, David Blaikie wrote:<br>
> On Tue, Dec 3, 2013 at 11:26 AM, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>><br>
> wrote:<br>
><br>
> > Can you please "svn up -r196270" and try running the R600 tests and add<br>
> > printfs to DwarfDebug::{begin,end}Function?<br>
> ><br>
> > e.g. see<br>
> > <a href="http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu/builds/10693" target="_blank">http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu/builds/10693</a><br>
> ><br>
> > I can guess that these tests don't actually need debug info (as they don't<br>
> > have any metadata),<br>
> > but it is obvious that DwarfDebug::endFunction() gets called without<br>
> > calling beginFunction() first.<br>
> ><br>
><br>
> Do we have an answer to this? Or a test case? etc?<br>
><br>
> Otherwise I'll be rather inclined to remove it as dead. Nothing exercises<br>
> it.<br>
><br>
<br>
</span>What are you planning to remove?<br></blockquote><div><br></div><div>This bit in particular:<br><br><span style="font-family:monospace">+  // Every beginFunction(MF) call should be followed by an endFunction(MF) call,</span><br style="font-family:monospace"><span style="font-family:monospace">+  // though the beginFunction may not be called at all.</span><br style="font-family:monospace"><span style="font-family:monospace">+  // We should handle both cases.</span><br style="font-family:monospace"><span style="font-family:monospace">+  if (CurFn == 0)</span><br style="font-family:monospace"><span style="font-family:monospace">+    CurFn = MF;</span><br style="font-family:monospace"><span style="font-family:monospace">+  else</span><br style="font-family:monospace"><span style="font-family:monospace">+    assert(CurFn == MF);</span><br></div><div> </div><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">
<span class=""><font color="#888888"><br>
-Tom<br>
</font></span><div class=""><div class="h5">><br>
><br>
> ><br>
> ><br>
> > 2013/12/3 Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>><br>
> ><br>
> >> On Tue, Dec 03, 2013 at 11:03:18PM +0400, Timur Iskhodzhanov wrote:<br>
> >> > 2013/12/3 Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>><br>
> >> ><br>
> >> > > > +  /// Please note that some AsmPrinter implementationss may not<br>
> >> call<br>
> >> > ><br>
> >> > > "implementations"<br>
> >> > ><br>
> >> ><br>
> >> > Oops! r196304<br>
> >> ><br>
> >> >  > +  /// beginFunction at all.<br>
> >> > ><br>
> >> > > Wat? They call endFunction but not beginFunction?<br>
> >> ><br>
> >> ><br>
> >> > +Tom<br>
> >> > You might want to revisit how AMDGPUAsmPrinter interacts with DwarfDebug<br>
> >> > through AsmPrinter.<br>
> >> ><br>
> >><br>
> >> Can you give me a little more background on the issue?  I don't know<br>
> >> anything about DwarfDebug and if R600 is using it, then it is probably<br>
> >> an accident.<br>
> >><br>
> >> -Tom<br>
> >><br>
> >> ><br>
> >> > > -eric<br>
> >> > ><br>
> >><br>
> >> > _______________________________________________<br>
> >> > llvm-commits mailing list<br>
> >> > <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> >> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
> >><br>
> >><br>
> ><br>
> > _______________________________________________<br>
> > llvm-commits mailing list<br>
> > <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
> ><br>
> ><br>
</div></div></blockquote></div><br></div></div>