Skip to content

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