<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 22, 2014 at 10:28 AM, Diego Novillo <span dir="ltr"><<a href="mailto:dnovillo@google.com" target="_blank" class="cremed">dnovillo@google.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 dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">
<div><div class="h5">On Wed, May 21, 2014 at 6:45 PM, Quentin Colombet <span dir="ltr"><<a href="mailto:qcolombet@apple.com" target="_blank" class="cremed">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">Hi Diego,<br>
<br>
Thanks for the diff, I was wondering where some of the changes came from :).<br>
<br>
This LGTM.<br>
<br>
One remark, would it be possible to move the PassRemarkXXX options and related implementation in the DiagnosticInfo implementation too?<br>
I.e., stuff related to static cl::opt<PassRemarksOpt, etc.<br>
<br>
If it is possible, you can fix that in a subsequent commit, if it is not I can live with that :).<br></blockquote><div><br></div></div></div><div>Thanks. Yeah, I think they're movable. I've committed this version and will send you a new patch that moves them shortly.</div>
</div></div></div></blockquote><div><br></div><div>Done.  <a href="http://reviews.llvm.org/D3878">http://reviews.llvm.org/D3878</a></div><div><br></div><div><br></div><div>Thanks. Diego.</div></div></div></div>