josiahyan edited a comment on pull request #8214:
URL: https://github.com/apache/arrow/pull/8214#issuecomment-696381588
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
josiahyan edited a comment on pull request #8214:
URL: https://github.com/apache/arrow/pull/8214#issuecomment-696381588
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
josiahyan edited a comment on pull request #8214:
URL: https://github.com/apache/arrow/pull/8214#issuecomment-696452697
@lidavidm @liyafan82 @jacques-n
Interpreting the results:
This patch could be improved (performance wise) by more aggressive caching
(option 3), at the potential
josiahyan edited a comment on pull request #8214:
URL: https://github.com/apache/arrow/pull/8214#issuecomment-696381588
@jacques-n I haven't done very much investigation on other speedups - I just
happened to notice performance irregularities as compared to our other (legacy)
codepaths,
josiahyan edited a comment on pull request #8214:
URL: https://github.com/apache/arrow/pull/8214#issuecomment-695114615
@liyafan82 Thanks for taking a look!
> I am a little surprised that JIT failed to transform the integer divistion
to a right shift, as it can be easily deduced
josiahyan edited a comment on pull request #8214:
URL: https://github.com/apache/arrow/pull/8214#issuecomment-695114615
@liyafan82 Thanks for taking a look!
> I am a little surprised that JIT failed to transform the integer divistion
to a right shift, as it can be easily deduced
