-
Notifications
You must be signed in to change notification settings - Fork 25.7k
make a way to disable callgrind #46116
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
Conversation
|
This pull request was exported from Phabricator. Differential Revision: D24227360 |
Codecov Report
@@ Coverage Diff @@
## master #46116 +/- ##
==========================================
- Coverage 68.26% 68.26% -0.01%
==========================================
Files 410 410
Lines 53533 53533
==========================================
- Hits 36543 36542 -1
- Misses 16990 16991 +1
Continue to review full report at Codecov.
|
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. Thanks for the fix.
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.
Thank you for the patch, but can you please extend it a bit and add a corresponding CMAKE option?
Summary: Pull Request resolved: pytorch#46116 Ideally I would just use one of the existing preprocessor flags such as `FBCODE_CAFFE2`, but this implies a whole bunch of other things elsewhere, so it is not really a solution for ovrsource. Test Plan: CI green, we are able to disable it internally with `-DNVALGRIND` Differential Revision: D24227360 fbshipit-source-id: 2145827bbb1fad22f4885b0c0a64352732c0de14
|
This pull request was exported from Phabricator. Differential Revision: D24227360 |
a99d0a0 to
5f781b5
Compare
|
This pull request has been merged in b1d24dd. |
Summary: Pull Request resolved: pytorch#46116 Ideally I would just use one of the existing preprocessor flags such as `FBCODE_CAFFE2`, but this implies a whole bunch of other things elsewhere, so it is not really a solution for ovrsource. Test Plan: CI green, we are able to disable it internally with `-DNVALGRIND` Reviewed By: malfet Differential Revision: D24227360 fbshipit-source-id: 24a3b393cf46d6a16acca0a9ec52610d4bb8704f (cherry picked from commit b1d24dd)
Summary: Ideally I would just use one of the existing preprocessor flags such as
FBCODE_CAFFE2, but this implies a whole bunch of other things elsewhere, so it is not really a solution for ovrsource.Test Plan: CI green, we are able to disable it internally with
-DNVALGRINDDifferential Revision: D24227360