Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DEV: Replace deprecated options for ifort.#5555

Merged
charris merged 1 commit into
numpy:masterfrom
ahjulstad:ifort-deprecated-options
Feb 11, 2015
Merged

DEV: Replace deprecated options for ifort.#5555
charris merged 1 commit into
numpy:masterfrom
ahjulstad:ifort-deprecated-options

Conversation

@ahjulstad
Copy link
Copy Markdown
Contributor

The /Qlowercase and /us options have been deprecated in the Intel Visual Fortran Compiler
since at least version 2013.
Replace with /names:lowercase and /assume:underscore.

Fixes #5554

The /Qlowercase and /us options has been deprecated in the Intel Visual Fortran Compiler since at least version 2013.
Replace with /names:lowercase and /assume:underscore.
@charris
Copy link
Copy Markdown
Member

charris commented Feb 10, 2015

Hmm, I don't think we can make this change if it breaks builds for people using older versions. How old a version will the new flags work with? Is it possible the make the flags version dependent?

@juliantaylor
Copy link
Copy Markdown
Contributor

intel removes flags from compilers with error after only two years? such an unimportant change should basically never be done ...
I don't think its a good idea to break builds on older compilers, especially as its a commercial one where upgrading may be difficult.

@ahjulstad
Copy link
Copy Markdown
Contributor Author

I only have 2013 and 2015 installed, so testing is limited, but I have found the new options in ifort documentation dated 2003. (Found it here.) It seems the options have been deprecated for while. But yes, it would have been better if ifort had thrown an error, instead of silently ignoring the options.

@charris
Copy link
Copy Markdown
Member

charris commented Feb 11, 2015

I've found other early references, so let's give this a try. Thanks @ahjulstad .

charris added a commit that referenced this pull request Feb 11, 2015
DEV: Replace deprecated options for ifort.
@charris charris merged commit 8cf7c4a into numpy:master Feb 11, 2015
@ahjulstad ahjulstad deleted the ifort-deprecated-options branch February 26, 2015 11:10
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.

Options for Intel Fortran Composer XE 2015 outdated

3 participants