<div dir="ltr">Looks like I argued (& then tested) previously for support for the case where the CU has no ranges, but sub-DIEs do: <a href="http://lists.llvm.org/pipermail/llvm-dev/2017-November/119131.html" target="_blank">http://lists.llvm.org/pipermail/llvm-dev/2017-November/119131.html</a><br><br>(Just for the record, LLVM gained support for CU ranges were implemented r<font color="#212121">197776, December 2013 (& shortly after that became the default in r</font><span style="color:rgb(33,33,33);font-size:13px">203968, March 2014 - in the 3.</span><span style="color:rgb(33,33,33)">5 release) - looks like GCC got this somewhere between GCC 4.1 and GCC 4.4 according to godbolt testing, so on/before March 2012 I think)</span><div dir="ltr"><font color="#212121"><br><br>So I've gone ahead and committed this change in r349333 - open to further discussion, reverting it, etc.<br><br>- Dave<br><br></font><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 28, 2018 at 10:48 AM David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yeah, fair - I'll give it a week or something, see if Paul or anyone else has ideas about why the existing behavior might be useful before I remove it.</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 28, 2018 at 10:27 AM Adrian Prantl <<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.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;line-break:after-white-space"><div><br><blockquote type="cite"><div>On Nov 28, 2018, at 9:50 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:</div><br class="m_4121663914027264793m_8327048114321083600m_-5074901734982490529m_839923426667656093Apple-interchange-newline"><div><div dir="ltr" style="font-family:Helvetica;font-size:12px;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;text-decoration:none"><br><div class="gmail_quote"><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">> a) decide that a unit without ranges covers no ranges - and don't do the search<br><br>Are there compilers that do this ("forget" to emit ranges) that we care to support with llvm-symbolizer?<br></blockquote><div><br>I'm not specifically aware of any, though haven't gone looking.<br></div></div></div></div></blockquote><div><br></div></div></div><div style="word-wrap:break-word;line-break:after-white-space"><div>Just in case this wasn't obvious in the sub-text:</div><div>I think we should figure out whether this assumption in llvm-symbolizer is actually needed to support a compiler we care about and then potentially remove it, or enforce it only when the CU is < DWARF 5 or something like that.</div></div><div style="word-wrap:break-word;line-break:after-white-space"><div><br></div><div>-- adrian</div></div></blockquote></div>
</blockquote></div></div></div>