<div dir="ltr">LGTM, please commit.<br><br>(I wouldn't mind it if there were some sense that this was a decision made by the Solaris platform - perhaps you are such a spokesperson for Solaris, but I don't know)</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 15, 2014 at 10:10 AM, Alexander Eremin <span dir="ltr"><<a href="mailto:alexander.r.eremin@gmail.com" target="_blank">alexander.r.eremin@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">Thanks for pointing, this is updated patch.<br><div class="gmail_extra"><br></div><div class="gmail_extra">Alex<br></div><span class=""><div class="gmail_extra"><br><div class="gmail_quote">2014-10-15 19:24 GMT+04:00 David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Sorry, I'm not sure I follow - what I meant is the patch needs (or should have, unless there's a reason this isn't testable, etc) a test case in Clang's regression suite. Probably in clang/test/CodeGen/debug-info-*<br><br>(you could try removing other targets from that conditional and see which tests fail - that should piont you to where to add tests for this platform)</div><div><div><div class="gmail_extra"><br></div></div></div></blockquote><div><br> </div></div></div></span></div>
</blockquote></div><br></div>