<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 3, 2013 at 8:26 PM, Quentin Colombet <span dir="ltr"><<a href="mailto:qcolombet@apple.com" target="_blank">qcolombet@apple.com</a>></span> wrote:<br>
<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">Hi Sean,<div><br></div><div>
Thanks for the pointer!</div><div><br></div><div>It would be sufficient, but it seems to require a large amount of work for this to be done properly.</div><div>See r174748.</div><div><br></div><div>In my opinion, the proposed patch can be seen as the first step towards this warning. When the proper diagnostic framework land in LLVM, this could be turned into the desired warning.</div>
<div><br></div><div>What do you think?</div><span class=""><font color="#888888"><div><br></div></font></span></div></blockquote><div><br></div><div>I think it is a reasonable first step, but as Evan points out, it is really more of a hack (dumping to errs() :| )than a proper solution. What are the next steps to evolving this feature into a proper solution, and are you guys planning on taking those steps? I don't think we want to just introduce a hack and leave it as such.</div>
<div><br></div><div style>-- Sean Silva</div></div></div></div>