[PATCH] D103135: [OpaquePtr] Remove existing support for forward compatibility

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

@t.p.northover seems to not be responding, can we go forward with this?

The motivation is that this implementation is assuming that `PointerType` will always have a pointee type. See

  /// Map all pointer types within \param Ty to the opaque pointer
  /// type in the same address space if opaque pointers are being
  /// used, otherwise nop. This converts a bitcode-reader internal
  /// type into one suitable for use in a Value.
  Type *flattenPointerTypes(Type *Ty) {
    return Ty;
  }

But we want to remove the pointee type in `PointerType` at some point.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103135