[cfe-commits] [PATCH] Driver: Unify CC1Options.td and Options.td, take 2!

Chandler Carruth chandlerc at google.com
Tue May 1 00:56:38 PDT 2012


On Tue, May 1, 2012 at 12:39 AM, James Molloy <james.molloy at arm.com> wrote:

> Hi Doug,
>
> Thanks! :) The only reason I haven't committed it yet given Eric's "LGTM"
> is
> that Chandler is currently reviewing it as a trial of the review board
> system here:
> http://llvm.org/reviews/r/7/
>
> When he OK's it, I'll commit it.
>

FYI, LGTM. I discussed the last few nit picks on IRC. Sorry for being slow.

Also, sorry to the list for the apparent silence on the review -- Anton is
holding back on enabling email for ReviewBoard until we're sure it's in a
plausible state.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120501/5be59632/attachment.html>


More information about the cfe-commits mailing list