<div dir="ltr">Per the Details:<div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><span style="color:rgb(0,0,0);font-family:"segoe ui","segoe ui web regular","segoe ui symbol",lato,"helvetica neue",helvetica,arial,sans-serif;font-size:13px">Prior to the change "DebugInfo: New metadata representation for global</span></div><div><span style="color:rgb(0,0,0);font-family:"segoe ui","segoe ui web regular","segoe ui symbol",lato,"helvetica neue",helvetica,arial,sans-serif;font-size:13px">variables." (</span><a href="https://reviews.llvm.org/D20147" class="gmail-phui-tag-view gmail-phui-tag-type-object" style="text-decoration:none;color:rgb(0,0,0);font-weight:bold;white-space:nowrap;font-family:"segoe ui","segoe ui web regular","segoe ui symbol",lato,"helvetica neue",helvetica,arial,sans-serif;font-size:13px"><span class="gmail-phui-tag-core-closed" style="text-decoration:line-through;color:rgba(0,0,0,0.498039);margin-top:0px"><span class="gmail-phui-tag-core gmail-phui-tag-color-object" style="color:rgb(0,0,0);border:1px solid rgb(231,231,231);border-radius:3px;padding:0px 4px;background-color:rgb(231,231,231);margin-top:0px">D20147</span></span></a><span style="color:rgb(0,0,0);font-family:"segoe ui","segoe ui web regular","segoe ui symbol",lato,"helvetica neue",helvetica,arial,sans-serif;font-size:13px">, </span><a href="https://reviews.llvm.org/D20415" class="gmail-phui-tag-view gmail-phui-tag-type-object" style="text-decoration:none;color:rgb(0,0,0);font-weight:bold;white-space:nowrap;font-family:"segoe ui","segoe ui web regular","segoe ui symbol",lato,"helvetica neue",helvetica,arial,sans-serif;font-size:13px"><span class="gmail-phui-tag-core-closed" style="text-decoration:line-through;color:rgba(0,0,0,0.498039);margin-top:0px"><span class="gmail-phui-tag-core gmail-phui-tag-color-object" style="color:rgb(0,0,0);border:1px solid rgb(231,231,231);border-radius:3px;padding:0px 4px;background-color:rgb(231,231,231);margin-top:0px">D20415</span></span></a><span style="color:rgb(0,0,0);font-family:"segoe ui","segoe ui web regular","segoe ui symbol",lato,"helvetica neue",helvetica,arial,sans-serif;font-size:13px">), clang emitted debug metadata for static</span></div><div><span style="color:rgb(0,0,0);font-family:"segoe ui","segoe ui web regular","segoe ui symbol",lato,"helvetica neue",helvetica,arial,sans-serif;font-size:13px">const values of floating-point types, but llvm did not translate this</span></div><div><span style="color:rgb(0,0,0);font-family:"segoe ui","segoe ui web regular","segoe ui symbol",lato,"helvetica neue",helvetica,arial,sans-serif;font-size:13px">debug metadata into DWARF. After that change, clang did not emit debug</span></div><div><span style="color:rgb(0,0,0);font-family:"segoe ui","segoe ui web regular","segoe ui symbol",lato,"helvetica neue",helvetica,arial,sans-serif;font-size:13px">metadata for static const values of floating-point types, but llvm is</span></div><div><span style="color:rgb(0,0,0);font-family:"segoe ui","segoe ui web regular","segoe ui symbol",lato,"helvetica neue",helvetica,arial,sans-serif;font-size:13px">capable of translating it into DWARF if it is emitted.</span></div></blockquote><font color="#000000" face="segoe ui, segoe ui web regular, segoe ui symbol, lato, helvetica neue, helvetica, arial, sans-serif"><br></font><div><font color="#000000" face="segoe ui, segoe ui web regular, segoe ui symbol, lato, helvetica neue, helvetica, arial, sans-serif">My (perhaps overly cautious) concern is that given what has occurred in the past (where llvm was not emitting bytes for floating-point types), it's possible that there might be a change in the future which would cause the emission of the bytes to become type-sensitive again, and I'd like to be sure we do continue to emit the bytes for floating-point types.</font></div><div><font color="#000000" face="segoe ui, segoe ui web regular, segoe ui symbol, lato, helvetica neue, helvetica, arial, sans-serif"><br></font></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 12, 2016 at 3:47 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><br><div class="gmail_quote"><span class=""><div dir="ltr">On Mon, Dec 12, 2016 at 3:24 PM David Gross 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dgross added a comment.<br class="m_-5303920041016513926gmail_msg">
<br class="m_-5303920041016513926gmail_msg">
In <a href="https://reviews.llvm.org/D27551#620373" rel="noreferrer" class="m_-5303920041016513926gmail_msg" target="_blank">https://reviews.llvm.org/<wbr>D27551#620373</a>, @dblaikie wrote:<br class="m_-5303920041016513926gmail_msg">
<br class="m_-5303920041016513926gmail_msg">
> Does this need the test coverage if it's just "hey, LLVM can splat these bytes out into DWARF even if we (independently) say the type of those bytes is a float"? The code that emits the bytes is presumably independent of the code that emits the type description, so testing the combination seems unnecessary to me.<br class="m_-5303920041016513926gmail_msg">
<br class="m_-5303920041016513926gmail_msg">
<br class="m_-5303920041016513926gmail_msg">
Can you explain this further?  What parts of this test case are you suggesting may be unnecessary?<br class="m_-5303920041016513926gmail_msg"></blockquote><div><br></div></span><div>All of it. (partly under the argument that no code changes were required to provide this functionality, but to be more complete:<br><br>* We already have test coverage for emitting the correct DW_AT_type value here<br>* We already have test coverage that the arbitrary bytes in the constant value are emitted into the right place in the DW_AT_const_value or whatnot<br><br>So what's the particularly interesting thing about testing the combination of those two (so far as I can see) orthogonal features?)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="m_-5303920041016513926gmail_msg">
<br class="m_-5303920041016513926gmail_msg">
<a href="https://reviews.llvm.org/D27551" rel="noreferrer" class="m_-5303920041016513926gmail_msg" target="_blank">https://reviews.llvm.org/<wbr>D27551</a><br class="m_-5303920041016513926gmail_msg">
<br class="m_-5303920041016513926gmail_msg">
<br class="m_-5303920041016513926gmail_msg">
<br class="m_-5303920041016513926gmail_msg">
</blockquote></div></div>
</blockquote></div><br></div>