<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 29, 2014 at 11:39 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"><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 dir="ltr"><<a href="mailto:glider@google.com" target="_blank">glider@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc 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: <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></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:0 0 0 .8ex;border-left:1px #ccc 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><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:0 0 0 .8ex;border-left:1px #ccc 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><br></div></div>