<div dir="ltr">Re-reverting this with r218716.<div><br></div><div>Kuba</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 30, 2014 at 2:39 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Mon, Sep 29, 2014 at 6:34 PM, Anna Zaks <span dir="ltr"><<a href="mailto:ganna@apple.com" target="_blank">ganna@apple.com</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"><div style="word-wrap:break-word"><br><div><div><div><blockquote type="cite"><div>On Sep 29, 2014, at 1:16 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:</div><br><div><br><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><div class="gmail_quote" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">On Mon, Sep 29, 2014 at 11:39 AM, Adrian Prantl<span> </span><span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>></span><span> </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"><div style="word-wrap:break-word"><br><div><span><blockquote type="cite"><div>On Sep 29, 2014, at 11:28 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:</div><br><div><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 29, 2014 at 6:17 AM, Alexander Potapenko<span> </span><span dir="ltr"><<a href="mailto:glider@google.com" target="_blank">glider@google.com</a>></span><span> </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>On Sat, Sep 27, 2014 at 2:22 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:<br>><br>><br>> On Fri, Sep 26, 2014 at 2:58 PM, Alexey Samsonov <<a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a>> wrote:<br>>><br>>> Author: samsonov<br>>> Date: Fri Sep 26 16:58:33 2014<br>>> New Revision: 218545<br>>><br>>> URL:<span> </span><a href="http://llvm.org/viewvc/llvm-project?rev=218545&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=218545&view=rev</a><br>>> Log:<br>>> Effectively revert r217284 on Darwin due to issues with -gmlt there<br>><br>><br>> Not sure if this is the right direction - but maybe I'm confused about what<br>> was discussed in the other thread.<br>><br>> I assume Apple uses/cares about -gmlt working on their platform and it<br>> currently doesn't. (I could be wrong though - maybe they don't care about<br>> -gmlt) This change would just paper over that.<br>><br>> But if -gmlt's behavior on Apple isn't important to anyone, I'm fine with<br>> this - keeps the code simpler on my side of things.<br>><br><br></span>Not sure about whether Apple cares about -gmlt (is that the new<br>preferred name for -gline-tables-only?) or not, but without this<br>change a bunch of asan tests has been failing on Darwin.<br></blockquote><div><br></div><div>Yeah, I'm just concerned this'll suppress the failure but leave the bug. But I'll leave it to Apple folks to decide what they care about/priorities/timelines when it comes to -gmlt.<br></div></div></div></div></div></blockquote><div><br></div></span><div>I’m a little confused now. Alexey, are the Darwin issues that you cited in the commit message the ones discussed in <span style="font-family:'Helvetica Neue'">r218129 (dsymutil skipping compile units without subprograms?) or is this a separate issue?<span> </span></span></div></div></div></blockquote><div><br></div><div>Yes, this is the same issue. This change causes the ASan buildbot to pass -g on darwin and -gline-tables-only on non-Darwin.</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"><div style="word-wrap:break-word"><div><div><span style="font-family:'Helvetica Neue'">We definitely do care about -gmlt working on Darwin, and we also care about Asan working on Darwin.</span></div></div></div></blockquote><div><br></div><div>Then you/I/we/someone should patch -gmlt to not do that optimization on Darwin & revert this patch to continue testing -gmlt + ASan.<br></div></div></div></blockquote><div><br></div></div></div><div>David,</div><div><br></div><div>We are interested in -gmlt + ASan working on Darwin.</div><div><br></div><div>Could you patch the optimization out on Darwin? Kuba will revert this patch after that’s done.</div></div></div></blockquote><div><br></div></div></div><div>This optimization is now disabled on Darwin by r218702. <br><br>Thanks!<span class="HOEnZb"><font color="#888888"><br>- David</font></span></div><span class=""><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"><div style="word-wrap:break-word"><div><div><br></div><div>Thank you!</div><span><font color="#888888"><div>Anna.</div></font></span><span><br><blockquote type="cite"><div><div class="gmail_quote" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><div><br>Well, maybe. It's not inherent that compiler-rt should be used to test -gmlt, but given that my understand was that -gmlt was implemented for ASan's convenience/use (and other things that just need backtracing/symbolication) I asked Alexey to change the compiler-rt testing to use -gmlt so I could make changes to it and be sure I wasn't regressing compiler-rt's needs. That seems as applicable to Darwin (if you guys care about -gmlt + ASan, specifically), otherwise you might want to setup some other bot with -gmlt + <whatever workload you care about being able to do stuff with -gmlt>, though I imagine it'll boil down to the same bugs caught in basically the same way.<br><br>- David</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"><div style="word-wrap:break-word"><div><span><font color="#888888"><div><span style="font-family:'Helvetica Neue'"><br></span></div><div><span style="font-family:'Helvetica Neue'">-- adrian</span></div></font></span></div></div></blockquote></div></div></blockquote></span></div><br></div></blockquote></span></div><br></div></div>
</blockquote></div><br></div>