Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

r2.6 cherry-pick request: Update ACL and oneDNN versions in mkl_aarch64 build #50772

Merged
merged 1 commit into from Jul 15, 2021

Conversation

penpornk
Copy link
Member

This cherry-pick:

  • Ensures the build flags are consistent on AArch64 between TF2.5 and 2.6
  • Provides a number of bug fixes via ACL which impact the functioning of the convolutions under certain conditions.
  • Fixes an issue with scaling ACL to high core counts (cost of spinning up many threads has been reduced).
  • Keeps the oneDNN version consistent with x86.

Original PR #50757 was merged yesterday (Tue 7/13) but not in time for the nightly branch cut. So we'll have to wait for Wed 7/14 nightly tests.

…nfig

Signed-off-by: Aleksandr Nikolaev <aleksandr.nikolaev@arm.com>
@google-cla
Copy link

google-cla bot commented Jul 14, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no label Jul 14, 2021
@penpornk
Copy link
Member Author

Manually setting CLA to yes because this commit is from PR #50708, which is already merged into master.

@penpornk penpornk added cla: yes and removed cla: no labels Jul 14, 2021
@gbaned gbaned self-assigned this Jul 15, 2021
@gbaned gbaned added the size:S CL Change Size: Small label Jul 15, 2021
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation Jul 15, 2021
@gbaned gbaned assigned mihaimaruseac and unassigned gbaned Jul 15, 2021
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Jul 15, 2021
@mihaimaruseac mihaimaruseac merged commit 766eec3 into tensorflow:r2.6 Jul 15, 2021
PR Queue automation moved this from Approved by Reviewer to Merged Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes size:S CL Change Size: Small
Projects
PR Queue
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

3 participants