Skip to content

Bugfix in concretize_compiler#2196

Merged
tgamblin merged 3 commits intospack:developfrom
RSE-Cambridge:bugfix-concretize
Nov 5, 2016
Merged

Bugfix in concretize_compiler#2196
tgamblin merged 3 commits intospack:developfrom
RSE-Cambridge:bugfix-concretize

Conversation

@js947
Copy link
Copy Markdown
Contributor

@js947 js947 commented Nov 1, 2016

The loop at lib/spack/spack/concretize.py:389 was not checking all of the elements in the list.

@davydden
Copy link
Copy Markdown
Member

davydden commented Nov 5, 2016

@alalazo @tgamblin : you probably want to have a look at this PR.

@tgamblin tgamblin merged commit a260e7e into spack:develop Nov 5, 2016
becker33 pushed a commit that referenced this pull request Nov 22, 2016
* fix bug in search for concrete compiler

* simplify implementation of concretize_compiler

* fix flake8 errors
@js947 js947 deleted the bugfix-concretize branch December 19, 2016 15:31
stephenmsachs pushed a commit to stephenmsachs/spack that referenced this pull request Nov 28, 2022
The Intel compiler isn't able to deal with noinline member functions of
template classses defined in headers.  As such it outputs
   warning spack#2196: routine is both "inline" and "noinline"
cmake bootstrap will fail due to the word 'warning'.

See spack/var/spack/repos/builtin/packages/protobuf/intel-v2.patch for reference.
stephenmsachs pushed a commit to stephenmsachs/spack that referenced this pull request Jan 3, 2023
Tagging @alalazo and @rschon2

The Intel compiler isn't able to deal with noinline member functions of template classses defined in headers. As such it outputs
warning spack#2196: routine is both "inline" and "noinline"
cmake bootstrap will fail due to the word 'warning'.

See spack/var/spack/repos/builtin/packages/protobuf/intel-v2.patch for reference.

The issue does not appear with [email protected] or later:

```
$~: compiler=/shared/spack/opt/spack/linux-amzn2-x86_64_v3/gcc-12.2.0/intel-oneapi-compilers-2022.2.0-uqvb2553zy5toeapvoopacndd27x6p5m/compiler/2022.2.0/linux/bin/intel64/icpc
$~: $compiler unique.c
icpc: remark spack#10441: The Intel(R) C++ Compiler Classic (ICC) is deprecated and will be removed from product release in the second half of 2023. The Intel(R) oneAPI DPC++/C++ Compiler (ICX) is the recommended compiler moving forward. Please transition to use this compiler. Use '-diag-disable=10441' to disable this message.
```

This is a clean version of spack#34167
alalazo pushed a commit that referenced this pull request Jan 5, 2023
The Intel compiler isn't able to deal with noinline member functions of template classses defined in headers. As such it outputs
```
warning #2196: routine is both "inline" and "noinline"
```
cmake bootstrap will fail due to the word 'warning'.

See spack/var/spack/repos/builtin/packages/protobuf/intel-v2.patch for reference.

The issue does not appear with [email protected] or later:

```
$~: compiler=/shared/spack/opt/spack/linux-amzn2-x86_64_v3/gcc-12.2.0/intel-oneapi-compilers-2022.2.0-uqvb2553zy5toeapvoopacndd27x6p5m/compiler/2022.2.0/linux/bin/intel64/icpc
$~: $compiler unique.c
icpc: remark #10441: The Intel(R) C++ Compiler Classic (ICC) is deprecated and will be removed from product release in the second half of 2023. The Intel(R) oneAPI DPC++/C++ Compiler (ICX) is the recommended compiler moving forward. Please transition to use this compiler. Use '-diag-disable=10441' to disable this message.
```

This is a clean version of #34167

Co-authored-by: Stephen Sachs <[email protected]>
RikkiButler20 pushed a commit to RikkiButler20/spack that referenced this pull request Jan 24, 2023
The Intel compiler isn't able to deal with noinline member functions of template classses defined in headers. As such it outputs
```
warning spack#2196: routine is both "inline" and "noinline"
```
cmake bootstrap will fail due to the word 'warning'.

See spack/var/spack/repos/builtin/packages/protobuf/intel-v2.patch for reference.

The issue does not appear with [email protected] or later:

```
$~: compiler=/shared/spack/opt/spack/linux-amzn2-x86_64_v3/gcc-12.2.0/intel-oneapi-compilers-2022.2.0-uqvb2553zy5toeapvoopacndd27x6p5m/compiler/2022.2.0/linux/bin/intel64/icpc
$~: $compiler unique.c
icpc: remark spack#10441: The Intel(R) C++ Compiler Classic (ICC) is deprecated and will be removed from product release in the second half of 2023. The Intel(R) oneAPI DPC++/C++ Compiler (ICX) is the recommended compiler moving forward. Please transition to use this compiler. Use '-diag-disable=10441' to disable this message.
```

This is a clean version of spack#34167

Co-authored-by: Stephen Sachs <[email protected]>
amd-toolchain-support pushed a commit to amd-toolchain-support/spack that referenced this pull request Feb 16, 2023
The Intel compiler isn't able to deal with noinline member functions of template classses defined in headers. As such it outputs
```
warning spack#2196: routine is both "inline" and "noinline"
```
cmake bootstrap will fail due to the word 'warning'.

See spack/var/spack/repos/builtin/packages/protobuf/intel-v2.patch for reference.

The issue does not appear with [email protected] or later:

```
$~: compiler=/shared/spack/opt/spack/linux-amzn2-x86_64_v3/gcc-12.2.0/intel-oneapi-compilers-2022.2.0-uqvb2553zy5toeapvoopacndd27x6p5m/compiler/2022.2.0/linux/bin/intel64/icpc
$~: $compiler unique.c
icpc: remark spack#10441: The Intel(R) C++ Compiler Classic (ICC) is deprecated and will be removed from product release in the second half of 2023. The Intel(R) oneAPI DPC++/C++ Compiler (ICX) is the recommended compiler moving forward. Please transition to use this compiler. Use '-diag-disable=10441' to disable this message.
```

This is a clean version of spack#34167

Co-authored-by: Stephen Sachs <[email protected]>
techxdave pushed a commit to Tech-XCorp/spack that referenced this pull request Feb 17, 2023
The Intel compiler isn't able to deal with noinline member functions of template classses defined in headers. As such it outputs
```
warning spack#2196: routine is both "inline" and "noinline"
```
cmake bootstrap will fail due to the word 'warning'.

See spack/var/spack/repos/builtin/packages/protobuf/intel-v2.patch for reference.

The issue does not appear with [email protected] or later:

```
$~: compiler=/shared/spack/opt/spack/linux-amzn2-x86_64_v3/gcc-12.2.0/intel-oneapi-compilers-2022.2.0-uqvb2553zy5toeapvoopacndd27x6p5m/compiler/2022.2.0/linux/bin/intel64/icpc
$~: $compiler unique.c
icpc: remark spack#10441: The Intel(R) C++ Compiler Classic (ICC) is deprecated and will be removed from product release in the second half of 2023. The Intel(R) oneAPI DPC++/C++ Compiler (ICX) is the recommended compiler moving forward. Please transition to use this compiler. Use '-diag-disable=10441' to disable this message.
```

This is a clean version of spack#34167

Co-authored-by: Stephen Sachs <[email protected]>
jmcarcell pushed a commit to key4hep/spack that referenced this pull request Apr 13, 2023
The Intel compiler isn't able to deal with noinline member functions of template classses defined in headers. As such it outputs
```
warning spack#2196: routine is both "inline" and "noinline"
```
cmake bootstrap will fail due to the word 'warning'.

See spack/var/spack/repos/builtin/packages/protobuf/intel-v2.patch for reference.

The issue does not appear with [email protected] or later:

```
$~: compiler=/shared/spack/opt/spack/linux-amzn2-x86_64_v3/gcc-12.2.0/intel-oneapi-compilers-2022.2.0-uqvb2553zy5toeapvoopacndd27x6p5m/compiler/2022.2.0/linux/bin/intel64/icpc
$~: $compiler unique.c
icpc: remark spack#10441: The Intel(R) C++ Compiler Classic (ICC) is deprecated and will be removed from product release in the second half of 2023. The Intel(R) oneAPI DPC++/C++ Compiler (ICX) is the recommended compiler moving forward. Please transition to use this compiler. Use '-diag-disable=10441' to disable this message.
```

This is a clean version of spack#34167

Co-authored-by: Stephen Sachs <[email protected]>
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.

3 participants