[cfe-commits] r86962 - /cfe/trunk/tools/clang-cc/clang-cc.cpp

[Sebastian Redl]

Daniel Dunbar wrote:
> Author: ddunbar
> Date: Thu Nov 12 00:48:24 2009
> New Revision: 86962
>
> URL: http://llvm.org/viewvc/llvm-project?rev=86962&view=rev
> Log:
> clang-cc: -fixit is actually option, not an action, although its use with non
> -fsyntax-only is probably untested.
>   
Also, misleading and not meaningful. Some warnings fixit the code to 
something else semantically than the current code, e.g. i & 1 == 0 is 
fixed to (i & 1) == 0. If I do CodeGen+Fixit, I get the old semantics in 
the object code even while changing the source.

So perhaps we should turn -fixit into an action instead of relying on 
users not to do anything stupid with it.

Sebastian