[PATCH] D104151: [docs][OpaquePtr] Shuffle around the transition plan section

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 11 15:09:56 PDT 2021


aeubanks marked an inline comment as done.
aeubanks added inline comments.


================
Comment at: llvm/docs/OpaquePointers.rst:140-143
+* Figure out how to name overloaded intrinsics with pointer parameters
+
+  * See ``getMangledTypeStr()``
+
----------------
dblaikie wrote:
> Oh, yeah. I remember that one.
> 
> Are there cases you know of where we will need overloading based on pointee type? Or might it be a case of mangling opaque pointer types as opaque being sufficient?
So far I've found D61524 that needs to be done.
But I'm confused about your question. We won't have pointee types, so we won't be able to mangle based on pointee type, we can only mangle opaque pointers as opaque pointers (with their address space).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104151/new/

https://reviews.llvm.org/D104151



More information about the llvm-commits mailing list