Skip to content

[SYCL-MLIR] Fix merge in #8764 #8817

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Mar 28, 2023
Merged

[SYCL-MLIR] Fix merge in #8764 #8817

merged 1 commit into from
Mar 28, 2023

Conversation

whitneywhtsang
Copy link
Contributor

@whitneywhtsang whitneywhtsang commented Mar 28, 2023 •
edited
Loading 

mlir-sycl/lib/Conversion/SYCLToLLVM/SYCLToLLVM.cpp:2185:31: error: redeclaration of using declaration
      ConvertSYCLToLLVMPass>::ConvertSYCLToLLVMBase;
      ~~~~~~~~~~~~~~~~~~~~~~~~^
mlir-sycl/lib/Conversion/SYCLToLLVM/SYCLToLLVM.cpp:2181:15: note: previous using declaration
  using Base::Base;
              ^

@whitneywhtsang
[SYCL-MLIR] Fix merge in intel#8764
53aa8b6 
Signed-off-by: Tsang, Whitney <whitney.tsang@intel.com>
@whitneywhtsang whitneywhtsang added the sycl-mlir Pull requests or issues for sycl-mlir branch label Mar 28, 2023
@whitneywhtsang whitneywhtsang requested a review from etiotto as a code owner March 28, 2023 00:13
@whitneywhtsang whitneywhtsang self-assigned this Mar 28, 2023
@sommerlukas sommerlukas mentioned this pull request Mar 28, 2023
Merged
@whitneywhtsang whitneywhtsang merged commit 7b513b7 into intel:sycl-mlir Mar 28, 2023
@whitneywhtsang whitneywhtsang deleted the fixbuild branch March 28, 2023 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sommerlukas sommerlukas sommerlukas approved these changes

@victor-eds victor-eds Awaiting requested review from victor-eds

@etiotto etiotto Awaiting requested review from etiotto

Assignees

@whitneywhtsang whitneywhtsang

Labels
sycl-mlir Pull requests or issues for sycl-mlir branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@whitneywhtsang @sommerlukas