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

Issue #518: Add Loop Vectorization #777

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

nuernbergk
Copy link

@nuernbergk nuernbergk commented Jul 3, 2024

Implements #518: Loop Vectorization.
All ew-Binary and Unary Operations are implemented, as well as nested expressions of these.
Currently not supported are casts and aggregation.
The test case loop-vectorization.mlir corresponds to this daphne code.

Note that there are formatting changes in DaphneIrExector.cpp and Passes.h, as I used the .clang-format provided in #723.
If necessary, I can of course revert these.

@philipportner philipportner self-requested a review July 3, 2024 11:11
@nuernbergk
Copy link
Author

The build failed on the include of the SCF.h file, this might because of a typo (capitalization error). Fixed. Can't test on my machine because it worked there even before fixing.

@philipportner
Copy link
Collaborator

The build failed on the include of the SCF.h file, this might because of a typo (capitalization error). Fixed. Can't test on my machine because it worked there even before fixing.

Hi Nils, this works, I applied the same patch before when compiling the code on my machine. Thanks for pushing the fix!

@pdamme pdamme added the AMLS summer 2024 Student project for the Architecture of ML Systems lecture at TU Berlin (summer 2024). label Aug 4, 2024
@philipportner philipportner self-assigned this Aug 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AMLS summer 2024 Student project for the Architecture of ML Systems lecture at TU Berlin (summer 2024).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants