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

Fixing 2000 #2004

Merged
merged 3 commits into from Feb 26, 2016
Merged

Fixing 2000 #2004

merged 3 commits into from Feb 26, 2016

Conversation

chinz07
Copy link
Contributor

@chinz07 chinz07 commented Feb 25, 2016

This fixes #2000

@chinz07
Copy link
Contributor Author

chinz07 commented Feb 25, 2016

Since assertion_failed/assertion_failed_msg never returns and the compiler can not infer this fact anymore it might be a good idea to mark the function with HPX_ATTRIBUTE_NORETURN. Any objections?

@hkaiser
Copy link
Member

hkaiser commented Feb 25, 2016

Since assertion_failed/assertion_failed_msg never returns and the compiler can not infer this fact anymore it might be a good idea to mark the function with HPX_ATTRIBUTE_NORETURN. Any objections?

No objections.

@chinz07
Copy link
Contributor Author

chinz07 commented Feb 25, 2016

@hkaiser Could you please merge this PR?

@hkaiser
Copy link
Member

hkaiser commented Feb 25, 2016

@chinz07 I was assuming you'd add the HPX_ATTRIBUTE_NORETURN first, were you planning to do that?

@chinz07
Copy link
Contributor Author

chinz07 commented Feb 25, 2016

@hkaiser I have marked the functions in commit f0553bc.

@hkaiser
Copy link
Member

hkaiser commented Feb 25, 2016

Ok, I'll merge it once buildbot has cycled later today

hkaiser added a commit that referenced this pull request Feb 26, 2016
@hkaiser hkaiser merged commit 63aec76 into STEllAR-GROUP:master Feb 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Undefined reference to hpx::assertion_failed
2 participants