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
[Backends] Change command line memory options to kilobytes #2956
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
13e22ff
to
9a5b64f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jfix71 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
9a5b64f
to
47c90df
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jfix71 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jfix71 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
CI is broken because the
en2gr_cpu_partition_test
tries to set the CPU memory on the command line, but it seems that command line parameters don't work withuint64_t
on some platforms. I've hit this issue in the past, and I'm not sure why it's the case.