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

[James Molloy]

Hi,

Could someone please review this? :(

Cheers,

James

-----Original Message-----
From: James Molloy 
Sent: 19 April 2012 16:22
To: cfe-commits at cs.uiuc.edu
Subject: FW: [cfe-commits] [PATCH] Driver: Unify CC1Options.td and
Options.td, take 2!

Bump.

-----Original Message-----
From: cfe-commits-bounces at cs.uiuc.edu
[mailto:cfe-commits-bounces at cs.uiuc.edu] On Behalf Of James Molloy
Sent: 18 April 2012 14:42
To: cfe-commits at cs.uiuc.edu
Subject: [cfe-commits] [PATCH] Driver: Unify CC1Options.td and Options.td,
take 2!

Hi,

Now that the LLVM conference is out of the way I can get back to my other
project that took a bit of a backseat, sorting out the clang driver.

I sent this patch upstream at the start of February, and it was about to
complete code review and signoff when Chandler went away on vacation for
ages and then afterwards I had the conference so it's been gathering dust in
my patch queue for a good 2 months.

I've rebased it to ToT and would like to have it reviewed now please :)

Original message thread at
http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20120123/052089.h
tml , with the last message here:
http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20120123/052094.h
tml 

The original message says it better, but essentially this patch removes the
duplication between CC1Options.td and Options.td, and removes the namespace
separation. This is one step in the path of removing the option forwarding
from the driver for Clang and relying on CompilerInvocation for all option
[de]serialization.

Chandler had some review comments, and my responses to those still stand
(the main one being I'll provide a followup patch to move CC1-only options
from Options.td to CC1Options.td).

Cheers!

James
-------------- next part --------------
A non-text attachment was scrubbed...
Name: unify-cc1options.reduced.diff
Type: application/octet-stream
Size: 19093 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120423/4b5c7e76/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: unify-cc1options.full.diff
Type: application/octet-stream
Size: 103891 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120423/4b5c7e76/attachment-0001.obj>