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

[Nuno Lopes]

Yes, we've noticed that ccc-analyzer is now more powerful than ccc. Anyway 
ccc is powerful enough for me right now. I've been only trying to compile 
PHP (it now only fails in the link phase)..  (btw, I haven't run your static 
analyzer on the PHP code yet, but I'll do so shortly)
Nuno

----- Original Message -----
> Thanks Nuno!
>
> There are a bunch of other flag options that ccc-analyzer currently 
> processes (and discards) that ccc does not.  I discovered these when 
> running the static analyzer over Postgresql and Xcode.  These should  be 
> gradually added back to ccc, although some of them may require  actual 
> support in the clang driver at some point.
> On Jun 17, 2008, at 10:23 AM, Nuno Lopes wrote:
>
>> Author: nlopes
>> Date: Tue Jun 17 12:23:14 2008
>> New Revision: 52417
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=52417&view=rev
>> Log:
>> handle -arch and -isysroot correctly (like ccc-analyszer do).
>> Patch by Filipe Cabecinhas
>>
>> Modified:
>>    cfe/trunk/utils/ccc
>>
>> Modified: cfe/trunk/utils/ccc
>> URL: 
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/utils/ccc?rev=52417&r1=52416&r2=52417&view=diff
>>
>> = = = = = = = = 
>> ======================================================================
>> --- cfe/trunk/utils/ccc (original)
>> +++ cfe/trunk/utils/ccc Tue Jun 17 12:23:14 2008
>> @@ -111,10 +111,26 @@
>>         if arg[:5] in ['-std=']:
>>             compile_opts.append(arg)
>>
>> +        # Options with one argument that should pass through to 
>> compiler
>> +        if arg in [ '-include', '-idirafter', '-iprefix',
>> +                    '-iquote', '-isystem', '-iwithprefix',
>> +                    '-iwithprefixbefore']:
>> +            compile_opts.append(arg)
>> +            compile_opts.append(args[i+1])
>> +            i += 1
>> +
>>         # Options with one argument that should pass through
>> -        if arg in ['-include', '-isysroot', '-arch']:
>> +        if arg in ['-framework']:
>> +            link_opts.append(arg)
>> +            link_opts.append(args[i+1])
>> +            i += 1
>> +
>> +        # Options with one argument that should pass through to both
>> +        if arg in ['-isysroot', '-arch']:
>>             compile_opts.append(arg)
>>             compile_opts.append(args[i+1])
>> +            link_opts.append(arg)
>> +            link_opts.append(args[i+1])
>>             i += 1
>>
>>         # Prefix matches for the link mode
>> @@ -123,12 +139,6 @@
>>             if arg == '-Os': arg = '-O2'
>>             link_opts.append(arg)
>>
>> -        # Options with one argument that should pass through
>> -        if arg in ['-framework', '-isysroot', '-arch']:
>> -            link_opts.append(arg)
>> -            link_opts.append(args[i+1])
>> -            i += 1
>> -
>>         # Input files
>>         if arg == '-filelist':
>>             f = open(args[i+1])
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits