<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">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:0 0 0 .8ex;border-left:1px #ccc 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>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><br></div><div><br></div><div>Diego.</div></div></div></div>