<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Sep 13, 2017 at 11:46 PM Dean Michael Berris <<a href="mailto:dberris@google.com">dberris@google.com</a>> wrote:<br></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"><div><div><blockquote type="cite"><div>On 13 Sep 2017, at 23:20, Dean Michael Berris <<a href="mailto:dberris@google.com" target="_blank">dberris@google.com</a>> wrote:</div><br class="m_-238116367201413111Apple-interchange-newline"><div><div style="font-family:HelveticaNeue;font-size:14px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><blockquote type="cite"><div><br class="m_-238116367201413111Apple-interchange-newline">On 13 Sep 2017, at 23:13, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:</div><br class="m_-238116367201413111Apple-interchange-newline"><div><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Sep 13, 2017 at 10:52 PM Dean Michael Berris via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">dberris updated this revision to Diff 115167.<br>dberris edited the summary of this revision.<br>dberris added reviewers: dblaikie, echristo.<br>dberris added a comment.<br>Herald added a subscriber: JDevlieghere.<br><br>Reworded description, added a test.<br><br><br><a href="https://reviews.llvm.org/D37791" rel="noreferrer" target="_blank">https://reviews.llvm.org/D37791</a><br><br>Files:<br>  lib/CodeGen/AsmPrinter/AsmPrinter.cpp<br>  test/DebugInfo/X86/xray-split-dwarf-interaction.ll<br><br><br>Index: test/DebugInfo/X86/xray-split-dwarf-interaction.ll<br>===================================================================<br>--- /dev/null<br>+++ test/DebugInfo/X86/xray-split-dwarf-interaction.ll<br>@@ -0,0 +1,76 @@<br>+; RUN: %llc_dwarf -split-dwarf-file=input.dwo -O3 -function-sections -data-sections \<br>+; RUN:     -relocation-model=pic -filetype=obj -generate-type-units -debug-compile -o %t %s<br></blockquote><div><br>Have you tried this without fission/split-dwarf? I imagine it still reproduces (I think it's only type units that are important?)<br> </div></div></div></div></blockquote><div><br></div><div>Not yet, but this was the case that caused the issues that I can reproduce. I can add another build here too.</div><br></div></div></blockquote><div><br></div></div></div></div><div style="word-wrap:break-word"><div><div><div>Actually, the "bug" wouldn't be fatal in that case, since we'd just have the wrong association. It's only when the debug section is stripped in case of split-dwarf that objcopy complains.</div></div></div></div></blockquote><div><br>Right, but what I mean is there's a bug here (the wrong association) even if it's not fatal. objcopy/split-dwarf exposed the underlying bug, but they aren't integral to the bug, they exacerbate it.<br><br>Though perhaps I'm misunderstanding what the 'association' is and what it being wrong might mean. Does that not result in any observable problems/incorrect behavior in xray when its wrong like this?<br> </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"><div><div>Landing this now, we can iterate more on it later.<br></div></div></div></blockquote><div><br></div><div>Generally once a patch is sent for review it shouldn't be committed until that review is approved. (the assumption being if something is sent for review it's because review is needed)<br><br>- Dave</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"><div><div></div><div><div><br></div><div>Thanks for the reviews!</div></div></div><br><br></div></blockquote></div></div>