Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

[v1.x] Backport of improve log_softmax op performance by using DNNL #18469

Merged
merged 1 commit into from Jun 3, 2020

Conversation

bgawrych
Copy link
Contributor

@bgawrych bgawrych commented Jun 2, 2020

Description

Backport of #18320 (Improve log_softmax op performance by using DNNL support)

@mxnet-bot
Copy link

Hey @bgawrych , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [centos-gpu, windows-gpu, windows-cpu, edge, unix-gpu, sanity, clang, website, miscellaneous, centos-cpu, unix-cpu]


Note:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.

…upport (apache#18320)

* Improve log_softmax performance by OneDNN library

* Adapt tests for MKLDNN log_softmax

* Fix lint errors

* Fix indent and comments
@bgawrych
Copy link
Contributor Author

bgawrych commented Jun 2, 2020

Hey @pengzhao-intel @ciyongch @TaoLv,
as @pengzhao-intel ask cherry-pick is ready :)

Copy link
Contributor

@pengzhao-intel pengzhao-intel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pengzhao-intel pengzhao-intel merged commit a64825f into apache:v1.x Jun 3, 2020
ChaiBapchya pushed a commit to ChaiBapchya/mxnet that referenced this pull request Aug 15, 2020
…upport (apache#18320) (apache#18469)

* Improve log_softmax performance by OneDNN library

* Adapt tests for MKLDNN log_softmax

* Fix lint errors

* Fix indent and comments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants