Skip to content

Commit 2f83f0a

Browse files
author
Spencer Bryngelson
committed
fix: remove GITHUB_ACTIONS guard — CCE 19 IPA crash also hits local builds
1 parent 78ba220 commit 2f83f0a

1 file changed

Lines changed: 6 additions & 9 deletions

File tree

CMakeLists.txt

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -735,21 +735,18 @@ if (MFC_PRE_PROCESS)
735735
if(CMAKE_Fortran_COMPILER_ID STREQUAL "Cray")
736736
target_compile_options(pre_process PRIVATE -hfp0)
737737
# CCE 19.0.0 IPA ICE: bring_routine_resident SIGSEGV in optcg during
738-
# ipa_processing on GitHub Actions runners. Only manifests in CI (runner
739-
# memory layout triggers the nondeterministic crash). Guard with
740-
# GITHUB_ACTIONS so local Frontier builds keep full IPA performance.
741-
if (DEFINED ENV{GITHUB_ACTIONS})
742-
target_compile_options(pre_process PRIVATE -Oipa0)
743-
endif()
738+
# ipa_processing. Nondeterministic — any file can become the crash site
739+
# via cross-file inlining. Safe to disable IPA for the whole target
740+
# (CPU-only, no GPU device-call requirements). See PR #1286.
741+
target_compile_options(pre_process PRIVATE -Oipa0)
744742
endif()
745743
endif()
746744

747745
if (MFC_SIMULATION)
748746
MFC_SETUP_TARGET(TARGET simulation
749747
SOURCES "${simulation_SRCs}"
750748
MPI FFTW OpenACC OpenMP)
751-
# CCE 19.0.0 optcg ICE on GitHub Actions runners: IPA inlining triggers two
752-
# distinct crashes in optcg:
749+
# CCE 19.0.0 optcg ICE: IPA inlining triggers two distinct crashes:
753750
# m_bubbles_EL: castIsValid assertion (InstCombine/foldIntegerTypedPHI)
754751
# m_phase_change, m_mpi_common, m_start_up: bring_routine_resident SIGSEGV
755752
# GPU builds: apply -Oipa0 per-file to known crash sites. Whole-target
@@ -758,7 +755,7 @@ if (MFC_SIMULATION)
758755
# CPU builds: disable IPA for the whole target (safe — no GPU requirements,
759756
# and the crash is nondeterministic so any file can become the crash site).
760757
# See PR #1286.
761-
if (CMAKE_Fortran_COMPILER_ID STREQUAL "Cray" AND DEFINED ENV{GITHUB_ACTIONS})
758+
if (CMAKE_Fortran_COMPILER_ID STREQUAL "Cray")
762759
if (MFC_OpenACC OR MFC_OpenMP)
763760
set_source_files_properties(
764761
"${CMAKE_BINARY_DIR}/fypp/simulation/m_bubbles_EL.fpp.f90"

0 commit comments

Comments
 (0)