Skip to content

Update fir int#168

Merged
YanzhouTang merged 3 commits intomainfrom
update_fir_int
Oct 21, 2025
Merged

Update fir int#168
YanzhouTang merged 3 commits intomainfrom
update_fir_int

Conversation

@tancheng
Copy link
Copy Markdown
Contributor

Provide fir int kernel

  • Update fir kernel test accordingly
  • No alloc needed

@Jackcuii plz check the generated mapping and IR. It has II=5, which is easier for you to parsing. And I update the data type as integer, so you don't need to change types.

@YanzhouTang
Copy link
Copy Markdown
Collaborator

YanzhouTang commented Oct 21, 2025

NOTES: if we still encounter the vectorization, it is due to the pragma in https://github.com/tancheng/CGRA-Bench/blob/master/kernels/fir/fir.cpp Line37. The #pragma clang loop unroll_count(4) vectorize_width(4) could make -fno-vectorize not take effect.

@YanzhouTang YanzhouTang merged commit 3bc440f into main Oct 21, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cleanup enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[P0] Issue with the constant op [P0] Mapping Algorithm Fails with -O3 -fno-vectorize -fno-slp-vectorize -mllvm -force-vector-width=1 in testbench

2 participants