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.

Sign up for GitHub

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

Jump to bottom

Fix atomic<int> deleted function #53

Merged
merged 1 commit into from
Oct 3, 2020
Merged

Fix atomic<int> deleted function #53

merged 1 commit into from
Oct 3, 2020

Conversation

NeroBurner
Copy link
Contributor 

"std::atomic<int>::atomic(const std::atomic<int> &)" (declared at line 668 of "/usr/include/c++/7/atomic"),
required for copy that was eliminated, cannot be referenced -- it is a deleted functionC/C++(1815)

@NeroBurner
Fix atomic<int> deleted function
bf81750 
```
"std::atomic<int>::atomic(const std::atomic<int> &)" (declared at line 668 of "/usr/include/c++/7/atomic"),
required for copy that was eliminated, cannot be referenced -- it is a deleted functionC/C++(1815)
```
@NeroBurner
Copy link
Contributor Author

@aras-p I think the Windows runner is borked. It always throws the following error, even though I did not change anything remotely touching flat_hash_map

 D:\a\ClangBuildAnalyzer\ClangBuildAnalyzer\src\external\flat_hash_map\flat_hash_map.hpp(1344,24): error C2039: 'out_of_range': is not a member of 'std' (compiling source file ..\..\src\Analysis.cpp) [D:\a\ClangBuildAnalyzer\ClangBuildAnalyzer\projects\vs2019\ClangBuildAnalyzer.vcxproj]

@aras-p
Copy link
Owner

aras-p commented Oct 3, 2020

I think the Windows runner is borked

Yeah looks like Github Actions upgraded VS2019 studio version it has, which has changed STL headers layout or include dependencies a bit. flat_hash_map.hpp was missing <stdexcept> include.

@aras-p aras-p merged commit e4a52e6 into aras-p:master Oct 3, 2020
@NeroBurner NeroBurner deleted the atomic_warning branch October 7, 2020 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@NeroBurner @aras-p