This is an automated email from the ASF dual-hosted git repository.

maxyang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudberry.git


The following commit(s) were added to refs/heads/main by this push:
     new 1a6697ee4d fix coredum when use group by with jit enabled PS: It 
should be noted that the elementType of v_resnullp needs to be the same as the 
number of bytes occupied by v_group_id, otherwise llvm-assert will be triggered.
1a6697ee4d is described below

commit 1a6697ee4d8271bbee4cc4689a08f8fa98e155a0
Author: ZhangHuiGui <[email protected]>
AuthorDate: Wed Mar 1 21:44:14 2023 +0800

    fix coredum when use group by with jit enabled
    PS:
    It should be noted that the elementType of v_resnullp
    needs to be the same as the number of bytes occupied
    by v_group_id, otherwise llvm-assert will be triggered.
---
 src/backend/jit/llvm/llvmjit_expr.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/backend/jit/llvm/llvmjit_expr.c 
b/src/backend/jit/llvm/llvmjit_expr.c
index 2df6e9f78e..84c5f5980a 100644
--- a/src/backend/jit/llvm/llvmjit_expr.c
+++ b/src/backend/jit/llvm/llvmjit_expr.c
@@ -1894,7 +1894,7 @@ llvm_compile_expr(ExprState *state)
 
                                        /* Copy aggstate->group_id to the 
result */
                                        v_group_id_p = 
l_ptr_const(&aggstate->group_id,
-                                                                               
          l_ptr(LLVMInt32Type()));
+                                                                               
          l_ptr(TypeSizeT));
                                        v_group_id = LLVMBuildLoad(b, 
v_group_id_p, "v_group_id");
 
                                        /* and store result */
@@ -1913,7 +1913,7 @@ llvm_compile_expr(ExprState *state)
 
                                        /* Copy aggstate->gset_id to the result 
*/
                                        v_gset_id_p = 
l_ptr_const(&aggstate->gset_id,
-                                                                               
          l_ptr(LLVMInt32Type()));
+                                                                               
          l_ptr(TypeSizeT));
                                        v_gset_id = LLVMBuildLoad(b, 
v_gset_id_p, "v_gset_id");
 
                                        /* and store result */
@@ -1932,7 +1932,7 @@ llvm_compile_expr(ExprState *state)
 
                                        /* Copy tsstate->currentExprId to the 
result */
                                        v_currentExprId_p = 
l_ptr_const(&tsstate->currentExprId,
-                                                                               
          l_ptr(LLVMInt32Type()));
+                                                                               
          l_ptr(TypeSizeT));
                                        v_currentExprId = LLVMBuildLoad(b, 
v_currentExprId_p, "v_currentExprId");
 
                                        /* and store result */


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to