[PATCH] D116464: [SPIRV 5/6] Add LegalizerInfo, InstructionSelector and utilities
Aleksandr Bezzubikov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 20 08:57:16 PDT 2022
zuban32 added a reviewer: nikic.
zuban32 added inline comments.
================
Comment at: llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp:286
+ if (auto PType = dyn_cast<PointerType>(Ty)) {
+ Type *ElemType = PType->getPointerElementType();
+
----------------
nikic wrote:
> This is incompatible with opaque pointers (https://llvm.org/docs/OpaquePointers.html). Please remove this call or revert the patch if this is non-trivial.
What if we use PointerType::getNonOpaquePointerElementType() instead? IMO that should do the trick while SPIR-V has no support of opaque pointers yet so we're only oging non-opaque way here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116464/new/
https://reviews.llvm.org/D116464
More information about the llvm-commits
mailing list