[ 
https://issues.apache.org/jira/browse/ARROW-7017?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16964416#comment-16964416
 ] 

Wes McKinney commented on ARROW-7017:
-------------------------------------

We should see what is possible as far as generating a shared library from a 
bundle of LLVM IR. If Gandiva developed the functionality to 

* pre-compile IR for all of its built-in functions (on a single-function basis) 
* generate a shared library from this
* have a way to construct a callable kernel given input types using the 
generated shared library

then that seems like a pretty good way forward. I see having multiple 
implementations of the same thing long term as being undesirable. In fairness I 
think in several places we may have extra logic in casting that is not present 
in Gandiva, but it would be worth investing the effort to try to have single 
implementations of things with suitable options to opt in to optional 
behaviors. 

> [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++, C++ - Compute
>            Reporter: Francois Saint-Jacques
>            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)

Reply via email to