[llvm-commits] [llvm] r167635 - /llvm/trunk/utils/TableGen/CodeEmitterGen.cpp

[Chad Rosier]

Test case?  Also, does this fix the failing test case, mblaze_memory.s?

 Chad

On Nov 9, 2012, at 1:27 PM, Evandro Menezes <emenezes at codeaurora.org> wrote:

> Author: emenezes
> Date: Fri Nov  9 15:27:03 2012
> New Revision: 167635
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=167635&view=rev
> Log:
> Fix issue with invalid flat operand number
> 
> Avoid iterating over list of operands beyond the number of operands in it.
> 
> PS: this fixes issue with revision #167634.
> 
> 
> Modified:
>    llvm/trunk/utils/TableGen/CodeEmitterGen.cpp
> 
> Modified: llvm/trunk/utils/TableGen/CodeEmitterGen.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/CodeEmitterGen.cpp?rev=167635&r1=167634&r2=167635&view=diff
> ==============================================================================
> --- llvm/trunk/utils/TableGen/CodeEmitterGen.cpp (original)
> +++ llvm/trunk/utils/TableGen/CodeEmitterGen.cpp Fri Nov  9 15:27:03 2012
> @@ -140,9 +140,7 @@
>     while (NumberedOp < NumberOps &&
>            CGI.Operands.isFlatOperandNotEmitted(NumberedOp))
>       ++NumberedOp;
> -    // If this operand has not been found, ignore it.
> -    if (NumberedOp >= NumberOps)
> -      return;
> +
>     OpIdx = NumberedOp++;
>   }
> 
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits