Skip to content

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