Skip to content

Bump rocm-systems from 093b66c to c6da7be#4097

Closed
amd-hsivasun wants to merge 1 commit intomainfrom
users/amd-hsivasun/bump-rocm-systems-c6da7bee2b
Closed

Bump rocm-systems from 093b66c to c6da7be#4097
amd-hsivasun wants to merge 1 commit intomainfrom
users/amd-hsivasun/bump-rocm-systems-c6da7bee2b

Conversation

@amd-hsivasun
Copy link
Contributor

Bumps rocm-systems from 96e30b4 to c6da7be.

Commits
  • c6da7be [rocprofiler-compute] update project files (#4267)
  • b1ce581 [RCCL] Fix validation errors with all_reduce_bias (#3570)
  • 123cf75 [rocSHMEM] fix compilation on older compilers (#4248)
  • 982b3e7 Found misnamed unit test (#3928)
  • 8fc1552 ROCM-20280 - hipGraphAddMemcpyNode should validate the extents of srcArray and dstArray (#4093)
  • affb615 ROCM-1667 - Check for OpenCL ICD loader presence when BUILD_ICD is not set explicitly (#3994)
  • 5c111d9 [hip-tests] Fix memory leak on single device systems (#4242)
  • d80aed9 rocrtst: Skip accessing queue handle when running ASAN
  • f462df8 rocr: ASAN Fixes: Fix use-after-free in counted queue release path
  • 56fa7fc runtime: fix malloc(0) in PtrInfo thunk path when count is zero
  • fbb6972 error out and print warning when channels exceed max supported channels for warpSpeed (#3577)
  • Additional commits viewable in compare view

Copy link
Contributor

@geomin12 geomin12 left a comment

Choose a reason for hiding this comment

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

test failures :( - rocblas is a bug but cannot tell if test failures are flaky / new / already existing

@tony-davis
Copy link
Contributor

This PR has failures in rocBLAS tests because a change introduced in rocm-libraries changed the interaction between TheRock CI and rocBLAS. This is resolved in PR #4098 , which will bring that change in from rocm-libraries and get things back in sync.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants