<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 10, 2014 at 1:27 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, Oct 10, 2014 at 12:49:08PM -0700, David Blaikie wrote:<br>
> On Fri, Oct 10, 2014 at 12:44 PM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br>
><br>
> > 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>
> > ><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<br>
> > 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>
> > What are you planning to remove?<br>
> ><br>
><br>
> This bit in particular:<br>
><br>
> +  // Every beginFunction(MF) call should be followed by an endFunction(MF)<br>
> call,<br>
> +  // though the beginFunction may not be called at all.<br>
> +  // We should handle both cases.<br>
> +  if (CurFn == 0)<br>
> +    CurFn = MF;<br>
> +  else<br>
> +    assert(CurFn == MF);<br>
><br>
<br>
</div></div>That's fine with me.  If there are bugs in the R600 backend that is<br>
preventing some useful change let me know and I'll take a look at it.<br></blockquote><div><br></div><div>Removed this in r219694. We'll see what happens.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
-Tom<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> ><br>
> > -Tom<br>
> > ><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<br>
> > 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>
> ><br>
</div></div></blockquote></div><br></div></div>