[cfe-commits] r102230 - in /cfe/trunk: include/clang/Driver/ include/clang/Frontend/ lib/Frontend/ test/FixIt/

[Nick Lewycky]

Chris Lattner wrote:
>
> On Apr 23, 2010, at 6:30 PM, Nick Lewycky wrote:
>
>> Author: nicholas
>> Date: Fri Apr 23 20:30:46 2010
>> New Revision: 102230
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=102230&view=rev
>> Log:
>> Teach clang -fixit to modify files in-place, or -fixit=suffix to create new
>> files with the additional suffix in the middle.
>
> Nice, does this mean I can just build with something like CXX='clang -fixit' and get auto-inplace-fixits happening?  Can you find somewhere to mention this in the users manual?

Almost! Clang still returns a failure exit code if one of the 
diagnostics was an error, regardless of whether the problems were fixed. 
You're certainly right that this is what I'm aiming for. (Also, your 
build will eventually fail because clang doesn't do compilation in this 
mode. Do we want to combine the two?)

Thanks for reminding me about the manual. I'll update it as soon as it's 
ready.

Nick