Skip to content

[build][hexagon] remove unused variable#13291

Merged
mehrdadh merged 1 commit into
apache:mainfrom
cconvey:fix-unused-var-warning
Nov 4, 2022
Merged

[build][hexagon] remove unused variable#13291
mehrdadh merged 1 commit into
apache:mainfrom
cconvey:fix-unused-var-warning

Conversation

@cconvey

@cconvey cconvey commented Nov 4, 2022

Copy link
Copy Markdown
Contributor

Remove unused member variable in the SimulatorRPCChannel class. Fixes a clang warning.

Remove unused member variable in the `SimulatorRPCChannel` class.
Fixes a clang warning.
@tvm-bot

tvm-bot commented Nov 4, 2022

Copy link
Copy Markdown
Collaborator

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

@cconvey

cconvey commented Nov 4, 2022

Copy link
Copy Markdown
Contributor Author

@mehrdadh : Mind reviewing this? It removes a member variable introduced by one of your PRs (#11065). Looks like it was only used for logging, and that logging statement is gone now.

@mehrdadh mehrdadh merged commit e860884 into apache:main Nov 4, 2022
@cconvey cconvey deleted the fix-unused-var-warning branch November 6, 2022 15:21
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 10, 2022
Remove unused member variable in the `SimulatorRPCChannel` class.
Fixes a clang warning.
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
Remove unused member variable in the `SimulatorRPCChannel` class.
Fixes a clang warning.
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.

3 participants