
<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Sep 29, 2014, at 1:16 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="">dblaikie@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><br class="Apple-interchange-newline"><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><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; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On Mon, Sep 29, 2014 at 11:39 AM, Adrian Prantl<span class="Apple-converted-space"> </span><span dir="ltr" class=""><<a href="mailto:aprantl@apple.com" target="_blank" class="">aprantl@apple.com</a>></span><span class="Apple-converted-space"> </span>wrote:<br class=""><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;" class=""><br class=""><div class=""><span class=""><blockquote type="cite" class=""><div class="">On Sep 29, 2014, at 11:28 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank" class="">dblaikie@gmail.com</a>> wrote:</div><br class=""><div class=""><div dir="ltr" class=""><br class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Mon, Sep 29, 2014 at 6:17 AM, Alexander Potapenko<span class="Apple-converted-space"> </span><span dir="ltr" class=""><<a href="mailto:glider@google.com" target="_blank" class="">glider@google.com</a>></span><span class="Apple-converted-space"> </span>wrote:<br class=""><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 Sat, Sep 27, 2014 at 2:22 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank" class="">dblaikie@gmail.com</a>> wrote:<br class="">><br class="">><br class="">> On Fri, Sep 26, 2014 at 2:58 PM, Alexey Samsonov <<a href="mailto:vonosmas@gmail.com" target="_blank" class="">vonosmas@gmail.com</a>> wrote:<br class="">>><br class="">>> Author: samsonov<br class="">>> Date: Fri Sep 26 16:58:33 2014<br class="">>> New Revision: 218545<br class="">>><br class="">>> URL:<span class="Apple-converted-space"> </span><a href="http://llvm.org/viewvc/llvm-project?rev=218545&view=rev" target="_blank" class="">http://llvm.org/viewvc/llvm-project?rev=218545&view=rev</a><br class="">>> Log:<br class="">>> Effectively revert r217284 on Darwin due to issues with -gmlt there<br class="">><br class="">><br class="">> Not sure if this is the right direction - but maybe I'm confused about what<br class="">> was discussed in the other thread.<br class="">><br class="">> I assume Apple uses/cares about -gmlt working on their platform and it<br class="">> currently doesn't. (I could be wrong though - maybe they don't care about<br class="">> -gmlt) This change would just paper over that.<br class="">><br class="">> But if -gmlt's behavior on Apple isn't important to anyone, I'm fine with<br class="">> this - keeps the code simpler on my side of things.<br class="">><br class=""><br class=""></span>Not sure about whether Apple cares about -gmlt (is that the new<br class="">preferred name for -gline-tables-only?) or not, but without this<br class="">change a bunch of asan tests has been failing on Darwin.<br class=""></blockquote><div class=""><br class=""></div><div class="">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 class=""></div></div></div></div></div></blockquote><div class=""><br class=""></div></span><div class="">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';" class="">r218129 (dsymutil skipping compile units without subprograms?) or is this a separate issue?<span class="Apple-converted-space"> </span></span></div></div></div></blockquote><div class=""><br class=""></div><div class="">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 class=""> </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;" class=""><div class=""><div class=""><span style="font-family: 'Helvetica Neue';" class="">We definitely do care about -gmlt working on Darwin, and we also care about Asan working on Darwin.</span></div></div></div></blockquote><div class=""><br class=""></div><div class="">Then you/I/we/someone should patch -gmlt to not do that optimization on Darwin & revert this patch to continue testing -gmlt + ASan.<br class=""></div></div></div></blockquote><div><br class=""></div><div>David,</div><div><br class=""></div><div>We are interested in -gmlt + ASan working on Darwin.</div><div><br class=""></div><div>Could you patch the optimization out on Darwin? Kuba will revert this patch after that’s done.</div><div><br class=""></div><div>Thank you!</div><div>Anna.</div><br class=""><blockquote type="cite" class=""><div class=""><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; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><div class=""><br class="">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 class=""><br class="">- David</div><div class=""> </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;" class=""><div class=""><span class=""><font color="#888888" class=""><div class=""><span style="font-family: 'Helvetica Neue';" class=""><br class=""></span></div><div class=""><span style="font-family: 'Helvetica Neue';" class="">-- adrian</span></div></font></span></div></div></blockquote></div></div></blockquote></div><br class=""></body></html>