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

ccalltest.c: ifndef Intel compiler for complex int #18193

Merged
merged 1 commit into from
Aug 23, 2016

Conversation

ehsantn
Copy link
Contributor

@ehsantn ehsantn commented Aug 22, 2016

Complex int is GNU extension that Intel compiler does not
support currently. [edit: fixes #17145

@@ -292,6 +292,8 @@ typedef struct {
complex double r89;
} struct_huge4b;

// Intel compiler does not currently support complex int (which is GNU extension)
#ifndef __INTEL_COMPILER
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we have a _COMPILER_INTEL_ define from src/support/platform.h for uniformity with other compiler vendor checks

Complex int is GNU extension that Intel compiler does not
support currently. JuliaLang#17145
@ehsantn
Copy link
Contributor Author

ehsantn commented Aug 23, 2016

Thanks. _COMPILER_INTEL_ is used now.

@ViralBShah ViralBShah added the building Build system, or building Julia or its dependencies label Aug 23, 2016
@ViralBShah ViralBShah added this to the 0.5.x milestone Aug 23, 2016
@tkelman tkelman merged commit b94da94 into JuliaLang:master Aug 23, 2016
tkelman pushed a commit that referenced this pull request Aug 29, 2016
Complex int is GNU extension that Intel compiler does not
support currently. #17145
(cherry picked from commit b94da94)
mfasi pushed a commit to mfasi/julia that referenced this pull request Sep 5, 2016
Complex int is GNU extension that Intel compiler does not
support currently. JuliaLang#17145
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building Build system, or building Julia or its dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

File ccalltest.c needs ifdef
3 participants