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

[Polybench FP] Stencil ADI #9

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

[Polybench FP] Stencil ADI #9

wants to merge 4 commits into from

Conversation

rachitnigam
Copy link
Member

@rachitnigam rachitnigam commented Mar 30, 2021

Example PR for porting a polybench benchmark to use a fixed point implementation.
It uses the m4 preprocessor to define constants and types in the file.
This allows us to write one implementation that works for both bit types and
fix types.

Steps

  • Syntactic rewrite of C++ program to Dahlia
  • Dahlia compiler reports no errors
  • Dahlia compiler generates valid Calyx program
  • Calyx and Dahlia programs generate the same result
  • Calyx and C++ programs generate same result with ubit inputs
  • Calyx and C++ programs generate same result with fix inputs

@rachitnigam
Copy link
Member Author

This benchmark tries to compute 1/20 in one of the first statements and therefore does not work with C++ simulation (immediately gets a divide by zero error).

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.

1 participant