[cfe-commits] r86962 - /cfe/trunk/tools/clang-cc/clang-cc.cpp
Sebastian Redl
sebastian.redl at getdesigned.at
Thu Nov 12 05:39:42 PST 2009
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
More information about the cfe-commits
mailing list