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 fusesoc and vivado xsim #139

Closed
wants to merge 2 commits into from

Conversation

skokvermon
Copy link

No description provided.

@niwis
Copy link
Collaborator

niwis commented Aug 9, 2022

common_cells.core was updated in #141 and #143. Regarding the xsim typedef: could you elaborate on this change in a separate PR? afaik XSIM should be the correct define. Finally, if this needs to be changed, this should be done globally in all source files where it is used.

@niwis niwis closed this Aug 9, 2022
@skokvermon
Copy link
Author

Sorry for late reply. I confirm XSIM is not defined with vivado xsim simulator
macro to use is XILINX_SIMULATOR
This change has to be made in all pulp code...

official source is in UG900 page
https://docs.xilinx.com/r/2021.2-English/ug900-vivado-logic-simulation/Predefined-Macros

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

Successfully merging this pull request may close these issues.

2 participants