[PATCH] D104272: [OpaquePtr] Mangle intrinsics with opaque pointers arguments

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 15 16:06:42 PDT 2021


dblaikie added a comment.

> The patch looks mostly good, but I'm not sold on "op". Can we just use "p"?

Presumably then we wouldn't know whether to parse a type after the address space or not? Unless we only support this when everything is opaque pointers? (so there can be no confusion between opaque pointers or not)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104272



More information about the llvm-commits mailing list