[cfe-commits] r55630 - /cfe/trunk/utils/ccc

[Nuno Lopes]

Because clang doesn't support all languages. If you are trying to compile a 
.S file, you'll be better served with gcc :-)
With this fix I'm "almost" able to compile valgrind without setting any 
environment vars.

Nuno


----- Original Message -----
> Why add the default value for CCC_LANGUAGES? The
> way it is written if CCC_LANGUAGES is not defined it
> defaults to all, implicitly.
>
> - Daniel
>
> On Tue, Sep 2, 2008 at 3:27 AM, Nuno Lopes <nunoplopes at sapo.pt> wrote:
>
>> Author: nlopes
>> Date: Tue Sep  2 05:27:37 2008
>> New Revision: 55630
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=55630&view=rev
>> Log:
>> recognise .s/.S file extensions. add default value for CCC_LANGUAGES
>>
>> Modified:
>>    cfe/trunk/utils/ccc
>>
>> Modified: cfe/trunk/utils/ccc
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/utils/ccc?rev=55630&r1=55629&r2=55630&view=diff
>>
>>
>> ==============================================================================
>> --- cfe/trunk/utils/ccc (original)
>> +++ cfe/trunk/utils/ccc Tue Sep  2 05:27:37 2008
>> @@ -26,7 +26,7 @@
>>  CCC_ECHO = checkenv('CCC_ECHO','1')
>>  CCC_NATIVE = checkenv('CCC_NATIVE')
>>  CCC_FALLBACK = checkenv('CCC_FALLBACK')
>> -CCC_LANGUAGES = checkenv('CCC_LANGUAGES')
>> +CCC_LANGUAGES =
>> checkenv('CCC_LANGUAGES','c,c++,c-cpp-output,objective-c,objective-c++,objective-c-cpp-output')
>>  if CCC_LANGUAGES:
>>     CCC_LANGUAGES = set([s.strip() for s in CCC_LANGUAGES.split(',')])
>>
>> @@ -164,6 +164,10 @@
>>         return "objective-c++"
>>     elif extension == "mi":
>>         return "objective-c-cpp-output"
>> +    elif extension == "s":
>> +        return "assembler"
>> +    elif extension == "S":
>> +        return "assembler-with-cpp"
>>     else:
>>         return "unknown"
>>