Skip to content

Conversation

AlexandreSinger
Copy link
Contributor

The sanitized build test was testing the Odin build and not the parmys build (which is the default). Moved the test to Parmys.

Resolves #3191

The sanitized build test was testing the Odin build and not the parmys
build (which is the default). Moved the test to Parmys.
@github-actions github-actions bot added the infra Project Infrastructure label Aug 7, 2025
@AlexandreSinger
Copy link
Contributor Author

It looks like there is some issues with the Sanitized build and parmys. I may have to investigate.

I do not think the sanitized build should even be running Parmys since we should really be debugging our own code. I think we should make a special test suite for the sanitized build which will run only VPR (i.e. will only use pre-generated blif files). To discuss.

@AlexandreSinger AlexandreSinger changed the title [CI] Moved Sanitized Build Test from Odin to Parmys [WIP][CI] Moved Sanitized Build Test from Odin to Parmys Aug 17, 2025
@vaughnbetz
Copy link
Contributor

Right now there is an error if we try to compile parmys with the sanitizers; it's a .so error so I think it is likely that parmys doesn't have the right flags passed to it to build the sanitized version. @loglav03 : if you have time to take a look that would be great, and adding @AmirhosseinPoolad as he tends to be very good at sorting build issues like this. Goal is to fix the error and convert the basic sanitized run to be on parmys instead of odin II.
@haydar-c : until this is fixed, you'd need to add the flat-recon very small test to the odin basic tests to get sanitized runs on them.

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

Successfully merging this pull request may close these issues.

[CI] Move Sanitized Build from Odin to Yosys
3 participants