-
Notifications
You must be signed in to change notification settings - Fork 396
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
Fix build error caused by unused variable in OSX #7316
Fix build error caused by unused variable in OSX #7316
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.
Thank you for supporting the project, and congratulations on your first contribution! A project committer will shortly review your contribution. In the mean time, if you haven't had a chance please skim over the contribution guidelines which all pull requests must adhere to. If the ECA pull request check fails, have a look at the instructions for signing the ECA in the legal considerations section.
If you run into any problems our community will be happy to assist you in any way we can. There are a number of recommended ways to interact with the community. We encourage you to ask questions, or drop by to say hello.
https://github.com/eclipse/omr/blob/master/CONTRIBUTING.md#commit-guidelines The following commit message requirement is not met: |
771ef6f
to
47ff490
Compare
Remove unused local variable in fvtest which will cause build error in OSX Signed-off-by: ChungHsuanChen <chunghsuanchen7@gmail.com>
47ff490
to
c299895
Compare
Jenkins build amac |
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.
The test failure on x86-64 macOS is not caused by this PR. See #7181.
Remove unused local variable in fvtest which will cause build error
In omrtimeTest.cpp
In si.cpp
Changes