Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
P parabix-devel
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • sla489
  • parabix-devel
  • Merge requests
  • !3

Merged
Created Jun 30, 2024 by sla489@sla489Maintainer

remove mvmd_compress, hsimd_signmask, and bitblock_advance implementations

  • Overview 0
  • Commits 1
  • Pipelines 1
  • Changes 2

Removes overrides in the idisa_arm_builder for mvmd_compress, hdimd_signmask, and bitblock_advance.

These methods appear to have been copied from the AVX builder and modified to work on ARM. Unlike others that were implemented this way, none of these make any use of ARM-specific features and provide little benefit over the default IDISA_Builder implementations.

I took perf measurements before and after removing each method. Cycle counts did not change meaningfully after removing hsimd_signmask and mvmd_compress. There was a small improvement after removing bitblock_advance.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: remove_methods