[llvm-commits] [llvm] r58905 - /llvm/trunk/lib/Target/CppBackend/CPPBackend.cpp

[Bill Wendling]

On Nov 8, 2008, at 4:58 AM, Anton Korobeynikov wrote:

> Author: asl
> Date: Sat Nov  8 06:58:07 2008
> New Revision: 58905
>
> URL: http://llvm.org/viewvc/llvm-project?rev=58905&view=rev
> Log:
> StoreInst does not produce any result thus it's useless to create new
> variable for it. This greatly reduces amount of unused variables in
> llvm2cpp-generated code
>
> Modified:
>    llvm/trunk/lib/Target/CppBackend/CPPBackend.cpp
>
> Modified: llvm/trunk/lib/Target/CppBackend/CPPBackend.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/CppBackend/CPPBackend.cpp?rev=58905&r1=58904&r2=58905&view=diff
>
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- llvm/trunk/lib/Target/CppBackend/CPPBackend.cpp (original)
> +++ llvm/trunk/lib/Target/CppBackend/CPPBackend.cpp Sat Nov  8  
> 06:58:07 2008
> @@ -1269,7 +1269,7 @@
>     }
>     case Instruction::Store: {
>       const StoreInst* store = cast<StoreInst>(I);
> -      Out << "StoreInst* " << iName << " = new StoreInst("
> +      Out << " = new StoreInst("

Should you remove the "=" as well?

-bw
>