Skip to content

add -fallow-argument-mismatch option when building CP2K 7.1 or older with GCC 10.x#2223

Merged
Micket merged 1 commit intoeasybuilders:developfrom
boegel:20201108102614_new_pr_EgEWPOJewB
Nov 9, 2020
Merged

add -fallow-argument-mismatch option when building CP2K 7.1 or older with GCC 10.x#2223
Micket merged 1 commit intoeasybuilders:developfrom
boegel:20201108102614_new_pr_EgEWPOJewB

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Nov 8, 2020 •
edited
Loading

(created using eb --new-pr)

fix for:

/tmp/cp2k-7.1/exts/dbcsr/src/block/dbcsr_block_access.f90:435:61:

  362 |                  CALL ${nametype1}$axpy(nze, scale, block(1:nze), 1, &
      |                                        2
......
  435 |                     CALL ${nametype1}$axpy(nze, ${one1[n]}$, block(1), 1, &
      |                                                             1
Error: Element of assumed-shape or pointer array as actual argument at (1) cannot correspond to actual argument at (2)

see also cp2k/cp2k#1157

@boegel boegel mentioned this pull request Nov 8, 2020
Merged
1 task
@boegel boegel added this to the 4.3.2 milestone Nov 8, 2020
@boegel boegel requested a review from bartoldeman November 8, 2020 09:59
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Nov 8, 2020

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS CP2K-7.1-foss-2020b.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3570.doduo.os - Linux RHEL 8.2, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/623825bd95c5296f5474cc06164cd002 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Nov 8, 2020

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS CP2K-7.1-foss-2020b.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3404.kirlia.os - Linux centos linux 7.8.2003, x86_64, Intel(R) Xeon(R) Gold 6240 CPU @ 2.60GHz (cascadelake), Python 2.7.5
See https://gist.github.com/b18b2b3b4342d3177f3eb5edbfbf6669 for a full test report.

Micket
Micket approved these changes Nov 9, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

lgtm

@Micket Micket merged commit 8594e0b into easybuilders:develop Nov 9, 2020
@boegel boegel deleted the 20201108102614_new_pr_EgEWPOJewB branch November 9, 2020 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Micket Micket Micket approved these changes

@bartoldeman bartoldeman Awaiting requested review from bartoldeman

Assignees

No one assigned

Labels

enhancement

Projects

None yet

Milestone

4.3.2

Development

Successfully merging this pull request may close these issues.

2 participants

@boegel @Micket