[
https://issues.apache.org/jira/browse/ARROW-7017?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17116221#comment-17116221
]
Wes McKinney commented on ARROW-7017:
-------------------------------------
I think the path forward here is to refactor to utilize common implementations
of for both the LLVM IR and pre-compiled kernels. In other words, what is
currently in the gandiva/precompiled directory would be moved to some place
where we can arrange so that these implementations are translated to LLVM IR
for use in Gandiva, while available as inline C/C++ functions for use in
creating pre-compiled vectorized kernels. Having multiple implementations of
the scalar "unit of work" does not seem desirable
> [C++] Refactor AddKernel to support other operations and types
> --------------------------------------------------------------
>
> Key: ARROW-7017
> URL: https://issues.apache.org/jira/browse/ARROW-7017
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++
> Reporter: Francois Saint-Jacques
> Assignee: Wes McKinney
> Priority: Major
> Labels: analytics
>
> * Should avoid using builders (and/or NULLs) since the output shape is known
> a compute time.
> * Should be refatored to support other operations, e.g. Substraction,
> Multiplication.
> * Should have a overflow, underflow detection mode.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
