Skip to content

Commit

Permalink
Add missing compiler name for two Intel config stanzas (wrf-model#1326)
Browse files Browse the repository at this point in the history
TYPE: bug fix

KEYWORDS: build, hydro, intel, stanza

SOURCE: Ryan Cabell (NCAR)

DESCRIPTION OF CHANGES:
Problem:
Building coupled WRF/WRF Hydro models fails when choosing configuration INTEL (ifort/icc): HSW/BDW. 

Solution:
Add missing `ifort compiler with icc` text to ARCH line for the HSW/BDW configuration. For completeness, the KNL MIC
configure stanza was also missing the same information, and has been updated.

Neither of these stanzas are involved in the traditional WRF code build, so there is no impact on users of the default WRF
atmosphere model.

ISSUE: For use when this PR closes an issue.
Fixes wrf-model#1325 

LIST OF MODIFIED FILES: 
M       arch/configure.defaults

TESTS CONDUCTED: 
1. Modification causes WRF-Hydro build to complete successfully with the `INTEL (ifort/icc): HSW/BDW` configuration
2. Jenkins testing all positive
  • Loading branch information
rcabell authored Jan 8, 2021
1 parent 432f33d commit 0a9c2eb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions arch/configure.defaults
Original file line number Diff line number Diff line change
Expand Up @@ -1834,7 +1834,7 @@ RLFLAGS =
CC_TOOLS = $(SCC)

###########################################################
#ARCH Linux x86_64 ppc64le i486 i586 i686 #serial smpar dmpar dm+sm
#ARCH Linux x86_64 ppc64le i486 i586 i686, ifort compiler with icc #serial smpar dmpar dm+sm
#
DESCRIPTION = INTEL ($SFC/$SCC): HSW/BDW
DMPARALLEL = # 1
Expand Down Expand Up @@ -1878,7 +1878,7 @@ RLFLAGS =
CC_TOOLS = $(SCC)

###########################################################
#ARCH Linux KNL x86_64 ppc64le i486 i586 i686 #serial smpar dmpar dm+sm
#ARCH Linux KNL x86_64 ppc64le i486 i586 i686, ifort compiler with icc #serial smpar dmpar dm+sm
#
DESCRIPTION = INTEL ($SFC/$SCC): KNL MIC
DMPARALLEL = # 1
Expand Down

0 comments on commit 0a9c2eb

Please sign in to comment.