Skip to content

Conversation

@aelovikov-intel
Copy link
Contributor

Apparently, we perform some hacks around constant address space that I wasn't aware of. Make sure new address cast helpers are called consistently with the previous behavior.

Apparently, we perform some hacks around constant address space that I
wasn't aware of. Make sure new address cast helpers are called
consistently with the previous behavior.
Copy link
Contributor

@sarnex sarnex left a comment

Choose a reason for hiding this comment

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

devops lgtm

Copy link
Contributor

@sergey-semenov sergey-semenov left a comment

Choose a reason for hiding this comment

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

Tiny nitpick aside, LGTM.

@aelovikov-intel aelovikov-intel merged commit ead1579 into intel:sycl Nov 1, 2024
13 checks passed
@aelovikov-intel aelovikov-intel deleted the fix-cts-multi-ptr branch November 1, 2024 14:19
@KornevNikita
Copy link
Contributor

@aelovikov-intel hi, it looks like the test fails after turning on

@aelovikov-intel
Copy link
Contributor Author

@aelovikov-intel hi, it looks like the test fails after turning on

I verified it locally when developing this fix and it passed. Will look into the failue.

@aelovikov-intel
Copy link
Contributor Author

@aelovikov-intel hi, it looks like the test fails after turning on

I verified it locally when developing this fix and it passed. Will look into the failue.

The regression that I fixed here was resulting in a segmentation fault and that's been fixed. The current failures are of these form:

  generic_ptr alias. Core types
  -------------------------------------------------------------------------------
  /__w/llvm/llvm/khronos_sycl_cts/tests/multi_ptr/multi_ptr_explicit_conversions_core.cpp:41
  ...............................................................................
  
  /__w/llvm/llvm/khronos_sycl_cts/tests/multi_ptr/multi_ptr_explicit_conversions_core.cpp:41: FAILED:
  explicitly with message:
    This test case has been compile-time disabled.
  
  -------------------------------------------------------------------------------
  generic_ptr alias. fp16 type
  -------------------------------------------------------------------------------
  /__w/llvm/llvm/khronos_sycl_cts/tests/multi_ptr/multi_ptr_explicit_conversions_fp16.cpp:45
  ...............................................................................
  
  /__w/llvm/llvm/khronos_sycl_cts/tests/multi_ptr/multi_ptr_explicit_conversions_fp16.cpp:45: FAILED:
  explicitly with message:
    This test case has been compile-time disabled.
  
  -------------------------------------------------------------------------------
  generic_ptr alias. fp64 type
  -------------------------------------------------------------------------------
  /__w/llvm/llvm/khronos_sycl_cts/tests/multi_ptr/multi_ptr_explicit_conversions_fp64.cpp:45
  ...............................................................................
  
  /__w/llvm/llvm/khronos_sycl_cts/tests/multi_ptr/multi_ptr_explicit_conversions_fp64.cpp:45: FAILED:
  explicitly with message:
    This test case has been compile-time disabled.
  
  ===============================================================================
  test cases:   47 |   44 passed | 3 failed
  assertions: 3589 | 3586 passed | 3 failed

and I don't see how it can be related to the regression I had introduced and subsequently fixed by this PR.

@aelovikov-intel
Copy link
Contributor Author

It looks like it's failing on the code added in KhronosGroup/SYCL-CTS#935. Tagging @victor-eds to take a look into it.

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.

4 participants