Skip to content

[DRAFT] Add SILU (s8 to s32), RQSILU (s8 to s8) for Siracusa/GAP9#167

Draft
FilippoCordella wants to merge 11 commits intopulp-platform:develfrom
FilippoCordella:Generic-SiLU
Draft

[DRAFT] Add SILU (s8 to s32), RQSILU (s8 to s8) for Siracusa/GAP9#167
FilippoCordella wants to merge 11 commits intopulp-platform:develfrom
FilippoCordella:Generic-SiLU

Conversation

@FilippoCordella
Copy link

Add SILU (s8 to s32) and RQSILU (s8 to s8) support for Siracusa/GAP9, including integration and tests.

Added

-SILU (s8 to s32) and RQSILU (s8 to s8) operators for Siracusa/GAP9.

Changed

Fixed

PR Merge Checklist

  1. The PR is rebased on the latest devel commit and pointing to devel.
  2. Your PR reviewed and approved.
  3. All checks are passing.
  4. The CHANGELOG.md file has been updated.
  5. If the docker was modified, change back its link after review.

Copy link
Contributor

@runwangdl runwangdl left a comment

Choose a reason for hiding this comment

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

Please check the comments.

DeeployTest/out.txt

CHANGELOG_GEN.md
TargetLibraries/PULPOpen/third_party/pulp-nn-mixed/
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this change. Run git submodule sync git submodule update --init --recursive


referenceTemplate = _SILUTemplate("""
// SILU (Name: ${nodeName}, Op: ${nodeOp})
SINGLE_CORE SILU_s${data_in_type.referencedType.typeWidth}_s${data_out_type.referencedType.typeWidth}(${data_in}, ${data_out}, ${size}, ${input_offset});
Copy link
Contributor

Choose a reason for hiding this comment

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

For the PULP platform, you should add a multicore template.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants