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

Add support for tiling of more FHELinalg operations #779

Merged
merged 8 commits into from
Apr 12, 2024

Conversation

andidr
Copy link
Contributor

@andidr andidr commented Apr 9, 2024

Add support for tiling of:

  • All element-wise ops
  • All flavors of applications of lookup tables in FHELinalg
  • All matmul operations
  • fhelinalg.sum
  • fhelinalg.transpose
  • fhelinalg.partition_frontier

andidr added 8 commits April 11, 2024 15:54
…inalg.generic

For now, the attribute "tile-sizes" is copied from the FHELinalg
operation to the corresponding `linalg.generic` operation only for
multiplications of encrypted matrices with plaintext matrices.

The changes of this commit also cause the attribute to be copied for
multiplications between plaintext ad ciphertext matrices, as well as
for multiplications between two ciphertext matrices.
…er tiling

The tiling infrastructure preserves attributes of tiled
`linalg.generic` operations, such that the attribute for the tile
sizes specified for the `linalg.generic` operation before tiling is
copied to the `linalg.generic` operation that is part of the generated
IR for a single tile.

This change causes the attribute to be removed after tiling, since it
does not make sense to preserve the attribute for per-tile operations.
@andidr andidr force-pushed the andi/tiling/additional-ops branch from ce9e5f2 to 1f56008 Compare April 11, 2024 13:54
@BourgerieQuentin BourgerieQuentin merged commit 2048105 into main Apr 12, 2024
27 of 29 checks passed
@BourgerieQuentin BourgerieQuentin deleted the andi/tiling/additional-ops branch April 12, 2024 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants