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

[SYCL][ESIMD] Change comparison method for for double arguments #1181

Merged
merged 1 commit into from
Aug 29, 2022
Merged

[SYCL][ESIMD] Change comparison method for for double arguments #1181

merged 1 commit into from
Aug 29, 2022

Conversation

fineg74
Copy link

@fineg74 fineg74 commented Aug 26, 2022

No description provided.

@fineg74 fineg74 requested a review from a team as a code owner August 26, 2022 01:03
@v-klochkov
Copy link

@fineg74
Copy link
Author

fineg74 commented Aug 29, 2022

Please verify the CI fail at bin_and_cmp_ops_heavy.cpp test here: http://icl-jenkins.sc.intel.com:8080/blue/organizations/jenkins/LLVM-Test-Suite-CI-TMP%2FLLVM-Test-Suite-CI-WIN/detail/LLVM-Test-Suite-CI-WIN/3085/pipeline

The only failing things I see are:

  1. Warning about deprecated getenv function in esimd_test_utils.hpp
  2. LLVM ERROR: GenX execution width and GRF crossing legalization failed for: < %.postcast = bitcast <8 x double> %call4.i.i.i36.i.i.esimd.i23.i.i.i to <16 x i32>>: 'double' data type is not supported by this target
    Both issues are not related to the PR.

@v-klochkov
Copy link

Linux is clean. THe windows fails are unrelated/known (and there is an internal tracker for that).

@v-klochkov v-klochkov merged commit 636b587 into intel:intel Aug 29, 2022
@fineg74 fineg74 deleted the BinTestUpdate branch August 30, 2022 03:54
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
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.

2 participants