<div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 23, 2012 at 4:14 AM, James Molloy <span dir="ltr"><<a href="mailto:james.molloy@arm.com" target="_blank">james.molloy@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
Could someone please review this? :(<br></blockquote><div><br></div><div>Yes, it is in my queue. Sadly, I haven't gotten to it (or much else in the way of Clang work) in the last few days, but it's still in the queue... Hopeful to get to it this week.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
<br>
James<br>
<div class="HOEnZb"><div class="h5"><br>
-----Original Message-----<br>
From: James Molloy<br>
Sent: 19 April 2012 16:22<br>
To: <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
Subject: FW: [cfe-commits] [PATCH] Driver: Unify CC1Options.td and<br>
Options.td, take 2!<br>
<br>
Bump.<br>
<br>
-----Original Message-----<br>
From: <a href="mailto:cfe-commits-bounces@cs.uiuc.edu">cfe-commits-bounces@cs.uiuc.edu</a><br>
[mailto:<a href="mailto:cfe-commits-bounces@cs.uiuc.edu">cfe-commits-bounces@cs.uiuc.edu</a>] On Behalf Of James Molloy<br>
Sent: 18 April 2012 14:42<br>
To: <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
Subject: [cfe-commits] [PATCH] Driver: Unify CC1Options.td and Options.td,<br>
take 2!<br>
<br>
Hi,<br>
<br>
Now that the LLVM conference is out of the way I can get back to my other<br>
project that took a bit of a backseat, sorting out the clang driver.<br>
<br>
I sent this patch upstream at the start of February, and it was about to<br>
complete code review and signoff when Chandler went away on vacation for<br>
ages and then afterwards I had the conference so it's been gathering dust in<br>
my patch queue for a good 2 months.<br>
<br>
I've rebased it to ToT and would like to have it reviewed now please :)<br>
<br>
Original message thread at<br>
<a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20120123/052089.h
tml" target="_blank">http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20120123/052089.h<br>
tml</a> , with the last message here:<br>
<a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20120123/052094.h
tml" target="_blank">http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20120123/052094.h<br>
tml</a><br>
<br>
The original message says it better, but essentially this patch removes the<br>
duplication between CC1Options.td and Options.td, and removes the namespace<br>
separation. This is one step in the path of removing the option forwarding<br>
from the driver for Clang and relying on CompilerInvocation for all option<br>
[de]serialization.<br>
<br>
Chandler had some review comments, and my responses to those still stand<br>
(the main one being I'll provide a followup patch to move CC1-only options<br>
from Options.td to CC1Options.td).<br>
<br>
Cheers!<br>
<br>
James</div></div></blockquote></div><br></div>