Skip to content

Conversation

@adamfidel
Copy link
Member

Current last remaining failures on PTL for SYCL Graph that we need to investigate further.

@adamfidel adamfidel requested a review from a team as a code owner August 26, 2025 18:42
@adamfidel adamfidel requested a review from mmichel11 August 26, 2025 18:42
Copy link
Contributor

@mmichel11 mmichel11 left a comment

Choose a reason for hiding this comment

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

LGTM, the CI failure appears unrelated.

@adamfidel
Copy link
Member Author

adamfidel commented Aug 28, 2025

@intel/llvm-gatekeepers This is ready to be merged. Thanks!
PVC errors in CI are unrelated.

  ********************
  Unexpectedly Passed Tests (1):
    SYCL :: Matrix/SG32/joint_matrix_bfloat16.cpp

@uditagarwal97
Copy link
Contributor

This commit will be authored by [email protected].

Hi @adamfidel ,
Could you please change your Github profile settings to allow committing using your work email? Please refer https://releases.llvm.org/21.1.0/docs/DeveloperPolicy.html#email-addresses

@adamfidel
Copy link
Member Author

@uditagarwal97 Thank you for the pointers. I changed my email settings. Do I need to force push a new commit for those changes to be reflected?

@uditagarwal97 uditagarwal97 merged commit 84114fe into intel:sycl Aug 29, 2025
26 of 27 checks passed
@uditagarwal97
Copy link
Contributor

@uditagarwal97 Thank you for the pointers. I changed my email settings. Do I need to force push a new commit for those changes to be reflected?

Nope, no need for force push.

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