Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL] add proper initialization for ze_kernel_properties_t inputs #460

Merged
merged 1 commit into from
Sep 14, 2021

Conversation

smaslov-intel
Copy link

Signed-off-by: Sergey V Maslov sergey.v.maslov@intel.com

Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
@v-klochkov
Copy link

@smaslov-intel - those fails in CI are unrelated to the fix, right?

@smaslov-intel
Copy link
Author

@smaslov-intel - those fails in CI are unrelated to the fix, right?

Yeah. I don't see how this could possibly be related.

@vladimirlaz
Copy link

the test failing was fixed in scope of #426

@vladimirlaz vladimirlaz merged commit 9a1832b into intel:intel Sep 14, 2021
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
Un-whitelist the test 5.0/target/test_target_allocate.c which has been
changed upstream in is now failing with

    test_target_allocate.c:27:32: error: allocator must be specified in the 'uses_allocators' clause
        #pragma omp target allocate(omp_default_mem_alloc:x) firstprivate(x) map(from: device_result)
                                    ^~~~~~~~~~~~~~~~~~~~~

Responsible upstream commit (SOLLVE/sollve_vv@f129ede):

    commit f129edeb595668056a7d117b0f00db40a05dd48e
    Author: Seyong Lee <lees2@ornl.gov>
    Date:   Fri Mar 11 12:10:04 2022 -0500

        Add new_test/test_target_allocate.F90. (intel#460)

        * Add new_test/test_target_allocate.F90.
        Test fails with both GCC11.1.0 and XLF16.1.1-10 (not supported).

        * add the default allocator expression to the allocate clause (allocate(omp_default_mem_alloc: x)) in both C and Fortran versions.
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants