diff --git a/FEXCore/Scripts/json_ir_generator.py b/FEXCore/Scripts/json_ir_generator.py index f3014f787..952263e2a 100755 --- a/FEXCore/Scripts/json_ir_generator.py +++ b/FEXCore/Scripts/json_ir_generator.py @@ -566,9 +566,7 @@ def print_ir_arg_printer(): SSAArgNum = 0 FirstArg = True - for i in range(0, len(op.Arguments)): - arg = op.Arguments[i] - + for arg in op.Arguments: # No point printing temporaries that we can't recover if arg.Temporary: continue @@ -683,22 +681,21 @@ def print_ir_allocator_helpers(): output_file.write("\tIRPair _{}(" .format(op.Name, op.Name)) # Output SSA args first - for i in range(0, len(op.Arguments)): - arg = op.Arguments[i] - LastArg = len(op.Arguments) - i - 1 == 0 + for i, arg in enumerate(op.Arguments): + LastArg = i == len(op.Arguments) - 1 if arg.Temporary: CType = IRTypesToCXX[arg.Type].CXXName - output_file.write("{} {}".format(CType, arg.Name)); + output_file.write("{} {}".format(CType, arg.Name)) elif arg.IsSSA: # SSA value output_file.write("OrderedNodeWrapper {}".format(arg.Name)) else: # User defined op that is stored CType = IRTypesToCXX[arg.Type].CXXName - output_file.write("{} {}".format(CType, arg.Name)); + output_file.write("{} {}".format(CType, arg.Name)) - if arg.DefaultInitializer != None: + if arg.DefaultInitializer: output_file.write(" = {}".format(arg.DefaultInitializer)) if not LastArg: @@ -756,20 +753,19 @@ def print_ir_allocator_helpers(): if op.SSAArgNum: output_file.write("\tIRPair _{}(" .format(op.Name, op.Name)) - for i in range(0, len(op.Arguments)): - arg = op.Arguments[i] - LastArg = len(op.Arguments) - i - 1 == 0 + for i, arg in enumerate(op.Arguments): + LastArg = i == len(op.Arguments) - 1 if arg.Temporary: CType = IRTypesToCXX[arg.Type].CXXName - output_file.write("{} {}".format(CType, arg.Name)); + output_file.write("{} {}".format(CType, arg.Name)) elif arg.IsSSA: output_file.write("OrderedNode *{}".format(arg.Name)) else: CType = IRTypesToCXX[arg.Type].CXXName - output_file.write("{} {}".format(CType, arg.Name)); + output_file.write("{} {}".format(CType, arg.Name)) - if arg.DefaultInitializer != None: + if arg.DefaultInitializer: output_file.write(" = {}".format(arg.DefaultInitializer)) if not LastArg: @@ -810,16 +806,15 @@ def print_ir_allocator_helpers(): print_validation(op) output_file.write(f"\t\treturn _{op.Name}(") - for i in range(0, len(op.Arguments)): - arg = op.Arguments[i] - LastArg = len(op.Arguments) - i - 1 == 0 + for i, arg in enumerate(op.Arguments): + LastArg = i == len(op.Arguments) - 1 output_file.write(arg.Name) if arg.IsSSA: output_file.write("->Wrapped(ListDataBegin)") if not LastArg: output_file.write(", ") - output_file.write(");\n"); - output_file.write("\t}\n\n"); + output_file.write(");\n") + output_file.write("\t}\n\n") output_file.write("#undef IROP_ALLOCATE_HELPERS\n") output_file.write("#endif\n")