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

[ESIMD] Add tests for ESIMDOptimizeVecArgCallConv pass. #1289

Merged
merged 3 commits into from
Sep 28, 2022

Conversation

kbobrovs
Copy link

Signed-off-by: Konstantin S Bobrovsky konstantin.s.bobrovsky@intel.com

Signed-off-by: Konstantin S Bobrovsky <konstantin.s.bobrovsky@intel.com>
@kbobrovs
Copy link
Author

/verify with intel/llvm#6889

// others.
// REQUIRES: gpu
// UNSUPPORTED: cuda || hip
// RUN: %clangxx -fsycl -Xclang -opaque-pointers -fsycl-device-only -Xclang -emit-llvm -o %t.comp.ll %s

Choose a reason for hiding this comment

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

You used '-opaque-pointers' here and '-opaque-ptr' for the 1st test above. Do both of spellings supported?

Copy link
Author

Choose a reason for hiding this comment

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

Thanks! No, just one, fixed.

// Dummy main to satisfy linker.
int main() {
return 0;
}

Choose a reason for hiding this comment

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

nit: please add new line

kbobrovs added a commit to intel/llvm that referenced this pull request Sep 27, 2022
…Conv.cpp (#6889)

Functions were erased from Module too early, while iterating over them.
Also fix signed/unsigned comparison warnings.

Tests: intel/llvm-test-suite#1289
This patch also fixes Linux failures.

Signed-off-by: Konstantin S Bobrovsky <konstantin.s.bobrovsky@intel.com>
@kbobrovs
Copy link
Author

/verify with intel/llvm#6889

@kbobrovs
Copy link
Author

/verify with intel/llvm#6889

@kbobrovs
Copy link
Author

verification with intel/llvm#6889 passed, #6889 is merged, merging.

@kbobrovs kbobrovs merged commit 18b3729 into intel:intel Sep 28, 2022
AlexeySachkov added a commit to AlexeySachkov/llvm that referenced this pull request Nov 16, 2022
Tests were added to intel/llvm-test-suite repo
(intel/llvm-test-suite#1289), but they are too low-level to be located
in there. Moved them in-tree.
AlexeySachkov added a commit to intel/llvm that referenced this pull request Nov 17, 2022
Tests were added to intel/llvm-test-suite repo
(intel/llvm-test-suite#1289), but they are too low-level to be located
in there. Moved them in-tree.
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
* [ESIMD] Add tests for ESIMDOptimizeVecArgCallConv pass.

Signed-off-by: Konstantin S Bobrovsky <konstantin.s.bobrovsky@intel.com>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…est-suite#1289)

* [ESIMD] Add tests for ESIMDOptimizeVecArgCallConv pass.

Signed-off-by: Konstantin S Bobrovsky <konstantin.s.bobrovsky@intel.com>
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