Gitiles
Code Review Sign In
review.mlplatform.org / ml / ComputeLibrary / 406a13f0b414d5c0375a46beec8dd9363a1cca56
commit406a13f0b414d5c0375a46beec8dd9363a1cca56[log] [tgz]
authorSiCong Li <sicong.li@arm.com>Wed Jul 15 12:09:58 2020 +0100
committerSiCong Li <sicong.li@arm.com>Tue Jul 21 09:41:49 2020 +0000
tree5f6fb7cfa1c7683d44de32840ffb541f450c8961
parentf6f7876e9ee8b58a8a6b335b032d554412fa3983 [diff]
COMPMID-3331 Remove y load padding from CLGEMMMatrixMultiplyReshapedOnlyRHSKernel and CLGEMMMatrixMultiplyNativeKernel

Resolves: COMPMID-3333, COMPMID-3334

* Implement an "overlap load, but don't overlap store" strategy:
    - Change STORE_BLOCK_BOUNDARY_AWARE so that the partial block in y
    dimension is placed at the beginning instead of at the end.
    - Implement 3 auxiliary functions to calculate the lhs, bias and dst
    addresses, taking into account the potential partial block in y dimension.
* Remove y load padding from Lhs and Bias tensors in
  CLGEMMMatrixMultiplyReshapedOnlyRHSKernel and CLGEMMMatrixMultiplyNativeKernel
* Modify config tests to assert zero-padding in new dimensions

Change-Id: I8f8585c7c0f543d720c2c91b885417c7dad35af4
Signed-off-by: SiCong Li <sicong.li@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/3576
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Reviewed-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
6 files changed
tree: 5f6fb7cfa1c7683d44de32840ffb541f450c8961
  1. .clang-format
  2. .clang-tidy
  3. .github/
  4. .gitmodules
  5. 3rdparty
  6. Android.bp
  7. LICENSE
  8. README.md
  9. SConscript
  10. SConstruct
  11. arm_compute/
  12. data/
  13. docs/
  14. examples/
  15. include/
  16. scripts/
  17. src/
  18. support/
  19. tests/
  20. utils/
README.md

Release repository: https://github.com/arm-software/ComputeLibrary

Development repository: https://review.mlplatform.org/#/admin/projects/ml/ComputeLibrary

Please report issues here: https://github.com/ARM-software/ComputeLibrary/issues

Make sure you are using the latest version of the library before opening an issue. Thanks

News:

Related projects:

Tutorials:

Documentation (API, changelogs, build guide, contribution guide, errata, etc.) available at https://github.com/ARM-software/ComputeLibrary/wiki/Documentation.

Binaries available at https://github.com/ARM-software/ComputeLibrary/releases.

License & Contributions: The software is provided under MIT license. Contributions to this project are accepted under the same license.

Public mailing list

For technical discussion, the ComputeLibrary project has a public mailing list: acl-dev@lists.linaro.org The list is open to anyone inside or outside of Arm to self subscribe. In order to subscribe, please visit the following website: https://lists.linaro.org/mailman/listinfo/acl-dev

Developer Certificate of Origin (DCO)

Before the ComputeLibrary project accepts your contribution, you need to certify its origin and give us your permission. To manage this process we use the Developer Certificate of Origin (DCO) V1.1 (https://developercertificate.org/)

To indicate that you agree to the the terms of the DCO, you "sign off" your contribution by adding a line with your name and e-mail address to every git commit message:

Signed-off-by: John Doe <john.doe@example.org>

You must use your real name, no pseudonyms or anonymous contributions are accepted.

Security Issues

If you believe you have discovered a security issue please contact MLG-Security@arm.com