[PATCH] D105321: [Bitcode][OpaquePtr] Remove usages of PointerType's getElementType()

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 7 15:04:30 PDT 2021


aeubanks added a comment.

> Tracking every Value* of pointer type seems expensive to me (but I'm just guessing), but maybe that's what it'd take.

Yeah it's pretty expensive, although that's what tnorthover's previous attempt did (which was in-tree for more than a year before I reverted it).
If we don't keep track of pointee types for bitcode that only uses opaque pointers, it shouldn't be noticeable for future bitcode that doesn't use typed pointers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105321



More information about the llvm-commits mailing list