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 macro to only call sched_getcpu on linux #177

Merged
merged 5 commits into from
Sep 30, 2019

Conversation

wu-haoze
Copy link
Collaborator

No description provided.

@guykatzz
Copy link
Collaborator

I'm not sure "common/CrossPlatform.h" is the place for this code - maybe just for the definition of a LINUX( ... ) macro. The rest of the code is very specific to DnC, so should be included within the DnC files. Or, alternatively, you can rename it to something more general (getCPUData, or something), and leave it under common.

@guykatzz
Copy link
Collaborator

LGTM

@wu-haoze wu-haoze merged commit b938c47 into NeuralNetworkVerification:master Sep 30, 2019
@wu-haoze wu-haoze deleted the log_cpu branch September 30, 2019 16:36
yuvaljacoby pushed a commit to yuvaljacoby/Marabou-1 that referenced this pull request Oct 7, 2019
…ation#177)

* add macro to only call sched_getcpu on linux

* cross platform get cpu id

* cosmetics and rename filename

* small fix
AleksandarZeljic pushed a commit to AleksandarZeljic/Marabou that referenced this pull request Oct 9, 2020
…ation#177)

* add macro to only call sched_getcpu on linux

* cross platform get cpu id

* cosmetics and rename filename

* small fix
matanost pushed a commit that referenced this pull request Nov 2, 2021
* add macro to only call sched_getcpu on linux

* cross platform get cpu id

* cosmetics and rename filename

* small fix
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