[PATCH] D125690: [MemIntrinsics] Add an API to get elementtype attribute value. NFC.

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 26 09:39:55 PDT 2022


anna added a comment.

In D125690#3538597 <https://reviews.llvm.org/D125690#3538597>, @aeubanks wrote:

> does using the size parameter not work for your use case? do you specifically need the type?

Unfortunately, size is not a precise way of differentiating primitives from array of pointers. We can still have the size of a primitive being the same as the size of an array (consider array with 1 element).
The specific use case for us is that when we lower these intrinsics, we need to know if we need to add the GC support for this (done in IR itself) or not. With opaque pointers without `elementttype` in the intrinsic, we lose this knowledge,


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125690



More information about the llvm-commits mailing list