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

Support for general data types in the frontend #49

Open
chhzh123 opened this issue May 12, 2022 · 2 comments
Open

Support for general data types in the frontend #49

chhzh123 opened this issue May 12, 2022 · 2 comments

Comments

@chhzh123
Copy link

Based on the example in samples folder, I found ScaleHLS can only support int and float types as inputs. If I try to use other types like bool and ap_int<N>, it will tell me the type name is unknown. Is this the limitation of Polygeist or simply not implemented by ScaleHLS?

@hanchenye
Copy link
Collaborator

Hi @chhzh123, I think you should be able to parse in bool type by including stdbool.h. Meanwhile, you'll need to set -I <location of your C headers> when calling mlir-clang. For ap_int<N>, because it's from Xilinx HLS library, I'm not sure whether there's a path to parse it in today's mlir-clang.

@chhzh123
Copy link
Author

chhzh123 commented May 13, 2022

Thanks for your reply! I actually noticed you had test cases for arbitrary integers and also had corresponding HLS codegen, so does that mean scalehls-opt can support those types if I directly code in MLIR instead of writing a C program?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants