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

James Molloy james.molloy at arm.com
Wed Apr 18 06:41:46 PDT 2012


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: 18630 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120418/d8093c9e/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: unify-cc1options.full.diff
Type: application/octet-stream
Size: 103161 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120418/d8093c9e/attachment-0001.obj>


More information about the cfe-commits mailing list