[
https://issues.apache.org/jira/browse/FLINK-4003?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15310841#comment-15310841
]
ASF GitHub Bot commented on FLINK-4003:
---------------------------------------
GitHub user greghogan opened a pull request:
https://github.com/apache/flink/pull/2059
[FLINK-4003] Use intrinsics for MathUtils logarithms
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/greghogan/flink
4003_use_intrinsics_for_mathutils_logarithms
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/2059.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2059
----
commit 46fc0d70ed7e7c0ea60ce60cf76f6ead996153cd
Author: Greg Hogan <[email protected]>
Date: 2016-06-01T16:29:16Z
[FLINK-4003] Use intrinsics for MathUtils logarithms
----
> Use intrinsics for MathUtils logarithms
> ---------------------------------------
>
> Key: FLINK-4003
> URL: https://issues.apache.org/jira/browse/FLINK-4003
> Project: Flink
> Issue Type: Improvement
> Components: Core
> Affects Versions: 1.1.0
> Reporter: Greg Hogan
> Assignee: Greg Hogan
> Priority: Trivial
> Fix For: 1.1.0
>
>
> {{MathUtils.log2floor}} and {{MathUtils.log2strict}} use naive loops which
> have efficient implementations in {{Integer}} that are commonly implemented
> as intrinsics [0].
> [0]:
> http://hg.openjdk.java.net/jdk8/jdk8/hotspot/file/tip/src/share/vm/classfile/vmSymbols.hpp#l680
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
