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

Conversation

maarquitos14
Copy link

#1583 introduced a clang-format issue. This fixes the issue.

Signed-off-by: Maronas, Marcos <marcos.maronas@intel.com>
@maarquitos14 maarquitos14 requested a review from a team as a code owner February 15, 2023 10:28
DataT *result_arr, DataT *result_scalar, sycl::queue &q) {
// We need to copy data_arr, otherwise when using a device it tries to use the host memory
// We need to copy data_arr, otherwise when using a device it tries to use the
// host memory

Choose a reason for hiding this comment

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

Suggested change
// host memory
// host memory.

To align with LLVM code style.

Signed-off-by: Maronas, Marcos <marcos.maronas@intel.com>
@steffenlarsen steffenlarsen merged commit 432cbbc into intel:intel Feb 17, 2023
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
…ntel#1596)

intel#1583 introduced a clang-format issue. This fixes the issue.

---------

Signed-off-by: Maronas, Marcos <marcos.maronas@intel.com>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…ntel/llvm-test-suite#1596)

intel#1583 introduced a clang-format issue. This fixes the issue.

---------

Signed-off-by: Maronas, Marcos <marcos.maronas@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.

3 participants