Remove duplicated return statement from llvmjit code.

anarazel · anarazel · commit 986070872ffa · 2018-06-26T23:16:50.000-07:00
The duplicated return clearly doesn't make sense / isn't reachable. Likely introduced by me (Andres), while revising the code. Author: Rushabh Lathia Discussion: https://postgr.es/m/CAGPqQf2raxWOcbuTP36M1rEF3=Rfo7oD29K3psdyHMeE5swBRg@mail.gmail.com
diff --git a/src/backend/jit/llvm/llvmjit.c b/src/backend/jit/llvm/llvmjit.c
@@ -394,7 +394,6 @@ llvm_function_reference(LLVMJitContext *context,
 		LLVMSetGlobalConstant(v_fn, true);
 
 		return LLVMBuildLoad(builder, v_fn, "");
-		return v_fn;
 	}
 
 	/* check if function already has been added */

Original file line number	Diff line number	Diff line change
`@@ -394,7 +394,6 @@ llvm_function_reference(LLVMJitContext *context,`
`394`	`394`	`LLVMSetGlobalConstant(v_fn, true);`
`395`	`395`
`396`	`396`	`return LLVMBuildLoad(builder, v_fn, "");`
`397`		`- return v_fn;`
`398`	`397`	`}`
`399`	`398`
`400`	`399`	`/* check if function already has been added */`