Skip to content

VS: Remove flag table entries for Fortran /Z* flags

Volker Jacht requested to merge kooky089/cmake:vs-fortran-debugging-flags into master

These were included when the Fortran flag table was first created by commit 10c91ded, but they map to fields not actually supported by the IDE. Remove their table entries.

Instead use just /debug:minimal and /debug:full to control debug information level in VS Intel Fortran. Let flags like /Z7 pass through as raw additional options because they have no corresponding IDE property and can be used to complement the supported options.

Original Description

cmLocalVisualStudio7Generator: Remove Fortran /Z* flags

@brad.king can you please review that slightly different approach? Instead of doing some extra treatment for /Z7, I thought those /Z flags could be directly passed to the additional options, as they are not supported by the VS Fortran IDE and they seem to be orthogonal to /debug:full and /debug:minimal.

Here are some references:

Intel Debug Windows:

https://software.intel.com/content/www/us/en/develop/documentation/fortran-compiler-developer-guide-and-reference/top/compiler-reference/compiler-options/compiler-option-details/output-debug-and-precompiled-header-pch-options/debug-windows.html

Intel Zi, Z7

https://software.intel.com/content/www/us/en/develop/documentation/fortran-compiler-developer-guide-and-reference/top/compiler-reference/compiler-options/compiler-option-details/output-debug-and-precompiled-header-pch-options/zi-z7.html

What do you think? Thanks.

From commit:

Use /debug:minimal and /debug:full to control debug information level in VS Intel Fortran. Remove /Z-flags as they are not supported by the IDE and can be used as complement to /debug:minimal /debug:full. /Zd is not supported by Intel Fortran (anymore?).

Fixes: #21340 (closed)
Backport: release

Edited by Brad King

Merge request reports

Loading