[cfe-commits] r72780 - in /cfe/trunk: include/clang/Driver/Options.def lib/Driver/Tools.cpp test/CodeGen/always_inline.c

[Daniel Dunbar]

On Wed, Jun 3, 2009 at 12:42 PM, Eli Friedman <eli.friedman at gmail.com> wrote:
> On Wed, Jun 3, 2009 at 11:25 AM, Daniel Dunbar<daniel at zuster.org> wrote:
>> On Wed, Jun 3, 2009 at 11:15 AM, Eli Friedman <eli.friedman at gmail.com> wrote:
>>> On Wed, Jun 3, 2009 at 9:16 AM, Daniel Dunbar<daniel at zuster.org> wrote:
>>>> Author: ddunbar
>>>> Date: Wed Jun  3 11:16:27 2009
>>>> New Revision: 72780
>>>>
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=72780&view=rev
>>>> Log:
>>>> Driver: Support -mllvm; this just forwards options to clang-cc.
>>>
>>> So, umm, we now have four different options that just forward stuff to
>>> clang-cc?  This seems like overkill...
>>
>> Four?
>
> -mllvm, -Xclang, -Xpreprocessor, and -Wp.  Oh, and also -Xanalyzer.

Ah. Well, -Xpreprocessor and -Wp, we inherit from gcc, and probably
should *not* forward to clang-cc and be handled internally. I'll kill
off -Xclang since -mllvm basically subsumes it. -Xanalyzer makes sense
(to me), since this is a different stage conceptually... originally I
added this in anticipation of ccc-analyzer using it, but if we create
a separate Driver client to replace ccc-analyzer then we can probably
remove it.

 - Daniel