[
https://issues.apache.org/jira/browse/ARROW-5272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16839090#comment-16839090
]
Pindikura Ravindra commented on ARROW-5272:
-------------------------------------------
[~pitrou] I tried this on my xenial setup (on GCE) with the same valgrind
settings, and wasn't able to reproduce this.
The travis build also didn't show failures in the decimal test
[https://travis-ci.org/apache/arrow/jobs/531878628]
Were you using some additional valgrind flags ?
> [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
> Assignee: Pindikura Ravindra
> Priority: Major
> Labels: pull-request-available
> Time Spent: 10m
> 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
(v7.6.3#76005)
