Skip to content
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

Fixup build failure with clang-11.0.0 #3836

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ojab
Copy link

@ojab ojab commented Nov 11, 2020

/tmp/ruby-build.20201111212834.9644.4pUxsX/rubinius-5.0/machine/diagnostics/timing.hpp:116:25: error: incompatible integer to pointer conversion assigning to 'std::atomic<uint64_t> *' (aka 'atomic<unsigned long> *') from 'uint64_t' (aka 'unsigned long')
      if(last_) last_ = run;
                        ^~~

Thank you for taking the time to submit a pull-request to Rubinius. We appreciate your contribution.

Please respond to the following questions to help us merge your pull-request. Please leave the form contents in place. If a question isn't relevant, please respond with N/A.

  1. Is this pull-request complete?
  • Yes, this pull-request is ready to be reviewed and merged.
  • No, this pull-request is a work-in-progress.
  1. Does this pull request fix an issue with an existing feature or introduce a new feature?
  • It fixes an issue with an existing features.
  • It introduces a new feature.
  1. Does this pull-request include tests?
  • Yes, it includes tests.
  • No, it does not include tests because the tests already exist.
  • No, it does not include tests because tests are too difficult to write.
  • No, it does not include tests because ...

```
/tmp/ruby-build.20201111212834.9644.4pUxsX/rubinius-5.0/machine/diagnostics/timing.hpp:116:25: error: incompatible integer to pointer conversion assigning to 'std::atomic<uint64_t> *' (aka 'atomic<unsigned long> *') from 'uint64_t' (aka 'unsigned long')
      if(last_) last_ = run;
                        ^~~
```
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.

None yet

2 participants