Skip to content

[SYCL][NFC] Remove multi_ptr<const void, Space>::operator multi_ptr<const void, Space> #1702

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.

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

Merged
merged 1 commit into from
May 18, 2020

Conversation

nliber
Copy link
Contributor

@nliber nliber commented May 17, 2020

It makes no sense (and won't be called) to have an implicit conversion operator to itself.

@nliber nliber requested a review from a team as a code owner May 17, 2020 23:44
@nliber nliber requested a review from s-kanaev May 17, 2020 23:44
@vladimirlaz
Copy link
Contributor

@v-klochkov, could you please have a look as you added this code?

@vladimirlaz vladimirlaz requested a review from v-klochkov May 18, 2020 06:04
@bader
Copy link
Contributor

bader commented May 18, 2020

@v-klochkov, could you please have a look as you added this code?

Looks like a no-brainer to me.

@bader bader changed the title [SYCL] Remove multi_ptr<const void, Space>::operator multi_ptr<const void, Space> [SYCL][NFC] Remove multi_ptr<const void, Space>::operator multi_ptr<const void, Space> May 18, 2020
@bader bader merged commit d2364a9 into intel:sycl May 18, 2020
alexbatashev pushed a commit to alexbatashev/llvm that referenced this pull request May 18, 2020
…_docs

* origin/sycl: (1867 commits)
  [SYCL] Implement USM vars and placeholder accessors passed to reduction (intel#1657)
  [SYCL] Fix post-commit testing (intel#1705)
  [SYCL][NFC] Remove multi_ptr<const void, Space>::operator multi_ptr<const void, Space> (intel#1702)
  [Doc] Readme update. (intel#1701)
  Support function pointers in cast instructions
  Adjust FPGA IVDep translation for embedded loops
  [SYCL] Fix host-to-host copy during copyback (intel#1692)
  [SYCL] Add information about dependencies used in CI
  [SYCL] Add runtime support for fsycl-id-queries-fit-in-int (intel#1685)
  Revert "Revert "[llvm][NFC] Cleanup uses of std::function in Inlining-related APIs""
  StoreInst should store Align, not MaybeAlign
  [clang][slh] Add test for SLH feature checking macro
  [NFC] Deduplicate comment in PromoteMemoryToRegister.cpp
  [WebAssembly] Optimize splats of bitcasted vectors
  [LLD][ELF] Use offset in thin archives to disambiguate thinLTO members
  [AArch64][SVE] Implement AArch64ISD::SETCC_PRED
  [SVE] Restore broken LLVM-C ABI compatability
  IR: Remove extra name mangling from llvm.ptrmask
  [NFC] Whitespace fix inside OptParserEmitter
  [compiler-rt][CMAKE] Only add cmake link flags in standalone build
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants