On Tue, Feb 5, 2013 at 10:12 AM, Eric Christopher <[email protected]>wrote:

>
>
>
> On Mon, Feb 4, 2013 at 11:47 PM, Chandler Carruth <[email protected]>wrote:
>
>>
>> On Mon, Feb 4, 2013 at 11:29 PM, Eric Christopher <[email protected]>wrote:
>>
>>> +  // -gsplit-dwarf should turn on -g and enable the backend dwarf
>>> +  // splitting and extraction.
>>> +  if (Args.hasArg(options::OPT_gsplit_dwarf)) {
>>> +    CmdArgs.push_back("-g");
>>> +    CmdArgs.push_back("-backend-option");
>>> +    CmdArgs.push_back("-split-dwarf=Enable");
>>>
>>
>> This seems a strange flag spelling... Are there values other than
>> "Enable"? I half expected just the standard binary '-split-debug' or
>> nothing of CC1 options...
>>
>
> Yeah, I put it as a tri-state Enable, Disable and Default so that we could
> enable it easier by default for certain targets.
>

That kind of defaulting logic seems better to have in the driver than in
CC1 to me.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to