[PATCH] D155086: [IR] Partially remove pointer element types from intrinsic signatures (NFC)

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 07:45:32 PDT 2023


nikic created this revision.
nikic added a reviewer: opaque-pointers.
Herald added subscribers: luke, mattd, gchakrabarti, pmatos, asb, StephenFan, frasercrmck, kerbowa, luismarques, apazos, sameer.abuasal, s.egerton, Jim, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, niosHD, sabuasal, simoncook, johnrusso, rbar, hiraditya, jgravelle-google, sbc100, jvesely, dschuff.
Herald added a project: All.
nikic requested review of this revision.
Herald added subscribers: llvm-commits, wangpc, jdoerfert, MaskRay, aheejin, jholewinski.
Herald added a project: LLVM.

As typed pointers are no longer supported, we should no longer specify element types in intrinsic signatures.

The only meaningful pointer types are now:

  llvm_ptr_ty -> ptr
  llvm_anyptr_ty -> ptr addrspace(any)
  LLVMQualPointerType<N> -> ptr addrspace(N)

This is only "partially" because we also have a bunch of special IIT descriptors like LLVMPointerTo, LLVMPointerToElt and LLVMAnyPointerToElt, which I'll leave for a later revision.


https://reviews.llvm.org/D155086

Files:
  llvm/include/llvm/IR/Intrinsics.td
  llvm/include/llvm/IR/IntrinsicsAArch64.td
  llvm/include/llvm/IR/IntrinsicsAMDGPU.td
  llvm/include/llvm/IR/IntrinsicsARM.td
  llvm/include/llvm/IR/IntrinsicsHexagon.td
  llvm/include/llvm/IR/IntrinsicsNVVM.td
  llvm/include/llvm/IR/IntrinsicsRISCV.td
  llvm/include/llvm/IR/IntrinsicsSystemZ.td
  llvm/include/llvm/IR/IntrinsicsWebAssembly.td
  llvm/include/llvm/IR/IntrinsicsX86.td
  llvm/lib/IR/Function.cpp
  llvm/test/TableGen/intrinsic-pointer-to-any.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D155086.539554.patch
Type: text/x-patch
Size: 58009 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230712/96112ff1/attachment-0001.bin>


More information about the llvm-commits mailing list