Skip to content

[6.38] Backport the fix for #21367 #22079

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.

Sign up for GitHub

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

Merged
hageboeck merged 2 commits into from
Apr 28, 2026
+6 −7
Merged

[6.38] Backport the fix for #21367 #22079

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4fa6268
[core] Disable one loop vectorisation to avoid a clang bug.
hageboeck Apr 24, 2026
cefbdf3
Revert "Disable the gl gradient tutorials on mac beta."
hageboeck Apr 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions core/base/src/TColorGradient.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,12 @@ void TColorGradient::ResetColor(UInt_t nPoints, const Double_t *points,
fColors.assign(colors, colors + nPoints * 4);

Double_t sum[4] = { 0., 0., 0., 0. };
#ifdef __clang__
// Clang vectoriser bug (confirmed for clang 20 - 22)
// https://github.com/llvm/llvm-project/issues/194008
// The store in the loop can smash the stack with e.g. -O2 -mavx2.
#pragma clang loop vectorize(disable)
#endif
for (unsigned n = 0; n < fColors.size(); ++n)
sum[n % 4] += fColors[n];

Expand Down
7 changes: 0 additions & 7 deletions tutorials/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -743,13 +743,6 @@ foreach(t ${tutorials})
if(${t} IN_LIST multithreaded)
set_tests_properties(tutorial-${tname} PROPERTIES PROCESSORS ${NProcessors})
endif()

if(CMAKE_SYSTEM_NAME MATCHES "Darwin" AND CMAKE_HOST_SYSTEM_VERSION VERSION_GREATER_EQUAL 25.4 AND tname MATCHES "gl-.*grad")
# ROOT-21367
# On mac beta, there is a stack corruption (suspected to be a compiler bug) that may fail these tests
set_tests_properties(tutorial-${tname} PROPERTIES DISABLED True)
message(NOTICE "Disabling tutorial ${tname} for mac beta")
endif()
endforeach()

#---Loop over all MPI tutorials and define the corresponding test---------
Expand Down
Loading