[ 
https://issues.apache.org/jira/browse/ARROW-5272?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Todd Farmer reassigned ARROW-5272:
----------------------------------

    Assignee:     (was: Pindikura Ravindra)

This issue was last updated over 90 days ago, which may be an indication it is 
no longer being actively worked. To better reflect the current state, the issue 
is being unassigned. Please feel free to re-take assignment of the issue if it 
is being actively worked, or if you plan to start that work soon.

> [C++] [Gandiva] JIT code executed over uninitialized values
> -----------------------------------------------------------
>
>                 Key: ARROW-5272
>                 URL: https://issues.apache.org/jira/browse/ARROW-5272
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++ - Gandiva
>            Reporter: Antoine Pitrou
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> When running Gandiva tests with Valgrind, I get the following errors:
> {code}
> [==========] Running 4 tests from 1 test case.
> [----------] Global test environment set-up.
> [----------] 4 tests from TestDecimal
> [ RUN      ] TestDecimal.TestSimple
> ==12052== Conditional jump or move depends on uninitialised value(s)
> ==12052==    at 0x41110D5: ???
> ==12052== 
> {
>    <insert_a_suppression_name_here>
>    Memcheck:Cond
>    obj:*
> }
> ==12052== Conditional jump or move depends on uninitialised value(s)
> ==12052==    at 0x41110E8: ???
> ==12052== 
> {
>    <insert_a_suppression_name_here>
>    Memcheck:Cond
>    obj:*
> }
> ==12052== Conditional jump or move depends on uninitialised value(s)
> ==12052==    at 0x411114B: ???
> ==12052== 
> {
>    <insert_a_suppression_name_here>
>    Memcheck:Cond
>    obj:*
> }
> ==12052== Conditional jump or move depends on uninitialised value(s)
> ==12052==    at 0x411117B: ???
> ==12052== 
> {
>    <insert_a_suppression_name_here>
>    Memcheck:Cond
>    obj:*
> }
> [       OK ] TestDecimal.TestSimple (16625 ms)
> [ RUN      ] TestDecimal.TestLiteral
> [       OK ] TestDecimal.TestLiteral (3480 ms)
> [ RUN      ] TestDecimal.TestIfElse
> [       OK ] TestDecimal.TestIfElse (2408 ms)
> [ RUN      ] TestDecimal.TestCompare
> [       OK ] TestDecimal.TestCompare (5303 ms)
> {code}
> I think this is legitimate. Gandiva runs computations over all values, even 
> when the bitmap indicates a null value. But decimal computations are complex 
> and involve conditional jumps, hence the error ("Conditional jump or move 
> depends on uninitialised value(s)").
> [~pravindra]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to