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

Jacques Nadeau commented on ARROW-3701:
---------------------------------------

Thanks for the explanations/pointers Antoine.

This suggest to me that we need to look more closely at when we use C++ as a 
basis for IR. In cases like decimal operations it seems like it might be better 
to write operations directly in LLVM ir since it seems like most of the 
operations are primitive instead of using front-end translation. Thoughts?
{quote}only the most common ones (8, 16, 32, 64) will be generally available
{quote}
Is this documented somewhere?

We may have to pick an optimal path for the common linux/modern cpus path and 
then a lowest common denominator fallback :(

 

 

> [Gandiva] Add support for decimal operations
> --------------------------------------------
>
>                 Key: ARROW-3701
>                 URL: https://issues.apache.org/jira/browse/ARROW-3701
>             Project: Apache Arrow
>          Issue Type: Task
>          Components: Gandiva
>            Reporter: Pindikura Ravindra
>            Assignee: Pindikura Ravindra
>            Priority: Major
>
> To begin with, will add support for 128-bit decimals. There are two parts :
>  # llvm_generator needs to understand decimal types (value, precision, scale)
>  # code decimal operations : add/subtract/multiply/divide/mod/..
>  ** This will be c++ code that can be pre-compiled to emit IR code



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to