Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Deprecate USE_PROFILER flag #15595

Merged
merged 2 commits into from
Jul 19, 2019
Merged

Deprecate USE_PROFILER flag #15595

merged 2 commits into from
Jul 19, 2019

Conversation

ChaiBapchya
Copy link
Contributor

@ChaiBapchya ChaiBapchya commented Jul 18, 2019

Description

Fixes #11563

Checklist

Essentials

Please feel free to remove inapplicable items for your PR.

  • Changes are complete (i.e. I finished coding on this PR)
  • To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

  • Removes USE_PROFILER flag from md, mk files across the codebase

@KellenSunderland

Copy link
Contributor

@KellenSunderland KellenSunderland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Many thanks for jumping on this one @ChaiBapchya.

@szha szha merged commit 6bdcef2 into apache:master Jul 19, 2019
@ChaiBapchya ChaiBapchya deleted the use_profiler_fix branch July 20, 2019 02:31
anirudhacharya pushed a commit to anirudhacharya/mxnet that referenced this pull request Aug 20, 2019
* deprecate USE_PROFILER

* Trigger notification
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecate USE_PROFILER from make/cmake.
4 participants