Skip to content

Add support to simd_bitmask#2677

Merged
celinval merged 10 commits intomodel-checking:mainfrom
celinval:issue-2131-bitmask
Aug 17, 2023
Merged

Add support to simd_bitmask#2677
celinval merged 10 commits intomodel-checking:mainfrom
celinval:issue-2131-bitmask

Conversation

@celinval
Copy link
Contributor

Description of changes:

Create a sequential model of simd_bitmask and create a transformation pass that replace calls to the simd_bitmask intrinsic by the new model.

We will only replace those usages that we can safely get information about the simd representation.

Resolved issues:

Resolves #2131

Related RFC:

Optional #ISSUE-NUMBER.

Call-outs:

  1. I still need to add some tests for the failure scenarios.
  2. I created Add flag to the compiler to build std library (no user visible change) #2676 to add the --build-std flag which is needed to build the standard library. Once that gets merged into main, all the related changes should go away.

Testing:

  • How is this change tested? new tests

  • Is this a refactor change?

Checklist

  • Each commit message has a non-empty body, explaining why the change was made
  • Methods or procedures are documented
  • Regression or unit tests are included, or existing tests cover the modified code
  • My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Create a sequential model of `simd_bitmask` and create a transformation
pass that replace calls to the simd_bitmask intrinsic by the new model.

We will only replace those usages that we can safely get information
about the simd representation.
@celinval celinval requested a review from a team as a code owner August 11, 2023 23:48
@celinval celinval added the Z-EndToEndBenchCI Tag a PR to run benchmark CI label Aug 11, 2023
@zhassan-aws
Copy link
Contributor

Thanks for implementing this, @celinval!

celinval and others added 3 commits August 15, 2023 16:36
Co-authored-by: Zyad Hassan <88045115+zhassan-aws@users.noreply.github.com>
@celinval celinval force-pushed the issue-2131-bitmask branch from 4b6858a to 724d163 Compare August 16, 2023 02:19
@celinval
Copy link
Contributor Author

Sorry, I merged before pulling from my remote branch, so I force pushed over 4b6858a.

Copy link
Contributor

@JustusAdam JustusAdam left a comment

Choose a reason for hiding this comment

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

Nice!

celinval and others added 3 commits August 17, 2023 10:59
@celinval celinval enabled auto-merge (squash) August 17, 2023 18:43
@celinval celinval merged commit db73dad into model-checking:main Aug 17, 2023
@zhassan-aws zhassan-aws mentioned this pull request Sep 12, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Z-EndToEndBenchCI Tag a PR to run benchmark CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support to simd_bitmask intrinsic

3 participants