-
Notifications
You must be signed in to change notification settings - Fork 607
Framework: Upgrade GCC #14709
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Framework: Upgrade GCC #14709
Conversation
|
CDash for AT1 results [Only accessible from Sandia networks] |
|
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request. |
|
Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Build InformationTest Name: PR_gcc-openmpi-openmp
Jenkins Parameters
Using Repos:
Pull Request Author: sebrowne |
e029b58 to
c10984b
Compare
|
Status Flag 'Pull Request AutoTester' - Error: Jenkins Jobs - A user has pushed a change to the PR before testing completed. NEW EVENT 'committed', ID C_kwDOAsJyMdoAKGMxMDk4NGI2MGIxZDE0ZDM2YjJhMmExMjRhYTViZGIyNTA1NjU3MTU... The Jenkins Jobs will be shutdown; Testing of this PR must occur again. |
|
Status Flag 'Pull Request AutoTester' - Jenkins Testing: 1 or more Jobs FAILED Note: Testing will normally be attempted again in approx. 2 Hrs 30 Mins. If a change to the PR source branch occurs, the testing will be attempted again on next available autotester run. Pull Request Auto Testing has FAILED (click to expand)Build InformationTest Name: PR_gcc-openmpi-openmp
Jenkins Parameters
|
|
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request. |
|
Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Build InformationTest Name: PR_gcc-openmpi-openmp
Jenkins Parameters
Using Repos:
Pull Request Author: sebrowne |
|
Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED Pull Request Auto Testing has PASSED (click to expand)Build InformationTest Name: PR_gcc-openmpi-openmp
Jenkins Parameters
|
|
Status Flag 'Pre-Merge Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging |
|
All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur... |
|
It seems like there are issues with git operations over http & ssh, hence why |
|
Hmmm looks like there are warning-as-error flags that were not valid for GCC10 but are valid for GCC12, leading to an increase in error output. I'm going to try disabling them one-by-one to get a summary of which ones are causing us trouble. |
|
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request. |
|
Status Flag 'Pull Request AutoTester' - GitHub reports Mergeable status = False |
Also add a new GCC12 + OpenMPI4 build. Because we have it named including the compiler version and it's marked as required I can't just delete it, so add the new one. Reduce the specificity of the versioning to make this easier (e.g. per-major-version) in the future. Signed-off-by: Samuel E. Browne <[email protected]>
|
@sebrowne There is a merge conflict. My PR for additional warnings-as-errors went in yesterday. |
3eeb717 to
5b60f7a
Compare
|
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request. |
|
Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Using Repos:
Pull Request Author: sebrowne |
|
Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED Pull Request Auto Testing has PASSED (click to expand) |
|
Status Flag 'Pre-Merge Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging |
|
All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur... |
5b60f7a to
017551f
Compare
|
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request. |
|
Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Using Repos:
Pull Request Author: sebrowne |
|
Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED Pull Request Auto Testing has PASSED (click to expand) |
|
Status Flag 'Pre-Merge Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging |
|
All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur... |
|
@trilinos/framework That's fun: Looks like the Zoltan2 test is now also failing in this PR, but in the OpenMP test that this PR doesn't change. I'm confused now: |
Those first two "Not Runs" were because the build failed due to the flag issues, so the files didn't get built for the test. This PR does change the OpenMP build to use the new toolchain. The title of that check isn't version-specific, so I changed it in-place. The "main" one is titled as gcc10, so we leave it and add a new one, switching over later. |
Appears to be a GCC bug causing `restrict` warnings, and some `maybe-uninitialized` ones that are more senstive. Downgrade them to warnings since getting the compiler upgraded is time-sensitive. Same with `redundant-move` and `mismatched-new-delete`. Signed-off-by: Samuel E. Browne <[email protected]>
017551f to
3b6511d
Compare
|
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request. |
|
Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Using Repos:
Pull Request Author: sebrowne |
|
Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED Pull Request Auto Testing has PASSED (click to expand) |
|
Status Flag 'Pre-Merge Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging |
|
All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur... |
2 similar comments
|
All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur... |
|
All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur... |
@trilinos/framework
Motivation
As mentioned in #14666, the new Kokkos + Trilinos exhibits errors with Intrepid2. Bump the main GCC image to GCC12 to avoid these issues.
Related Issues
https://sems-atlassian-son.sandia.gov/jira/browse/TRILFRAME-739
Testing
Letting the CI test this.