https://github.com/nikic created https://github.com/llvm/llvm-project/pull/183093
This drops one getPointerType() overload which accepted a name, which is no longer used since the opaque pointers migration. The fallback code path always returns a plain pointer now. Also drop all the virtual qualifiers. Nothing inherits from this class. Any customization is implemented via TargetCodeGenInfo hooks in the implementation. >From c10b5bf12a5922d1f181e6e1e7f6cab206903421 Mon Sep 17 00:00:00 2001 From: Nikita Popov <[email protected]> Date: Tue, 24 Feb 2026 17:32:35 +0100 Subject: [PATCH] [CGOpenCLRuntime] Remove dead code This drops one getPointerType() overload which accepted a name, which is no longer used since the opaque pointers migration. The fallback code path always returns a plain pointer now. Also drop all the virtual qualifiers. Nothing inherits from this class. Any customization is implemented via TargetCodeGenInfo hooks in the implementation. --- clang/lib/CodeGen/CGOpenCLRuntime.cpp | 8 -------- clang/lib/CodeGen/CGOpenCLRuntime.h | 22 +++++++++------------- 2 files changed, 9 insertions(+), 21 deletions(-) diff --git a/clang/lib/CodeGen/CGOpenCLRuntime.cpp b/clang/lib/CodeGen/CGOpenCLRuntime.cpp index 9f8ff488755ec..8c0b98e0d6f7b 100644 --- a/clang/lib/CodeGen/CGOpenCLRuntime.cpp +++ b/clang/lib/CodeGen/CGOpenCLRuntime.cpp @@ -53,14 +53,6 @@ llvm::Type *CGOpenCLRuntime::getPipeType(const PipeType *T) { if (llvm::Type *PipeTy = CGM.getTargetCodeGenInfo().getOpenCLType(CGM, T)) return PipeTy; - if (T->isReadOnly()) - return getPipeType(T, "opencl.pipe_ro_t", PipeROTy); - else - return getPipeType(T, "opencl.pipe_wo_t", PipeWOTy); -} - -llvm::Type *CGOpenCLRuntime::getPipeType(const PipeType *T, StringRef Name, - llvm::Type *&PipeTy) { if (!PipeTy) PipeTy = getPointerType(T); return PipeTy; diff --git a/clang/lib/CodeGen/CGOpenCLRuntime.h b/clang/lib/CodeGen/CGOpenCLRuntime.h index 34613c3516f37..bd7960c1ed6bc 100644 --- a/clang/lib/CodeGen/CGOpenCLRuntime.h +++ b/clang/lib/CodeGen/CGOpenCLRuntime.h @@ -36,8 +36,7 @@ class CodeGenModule; class CGOpenCLRuntime { protected: CodeGenModule &CGM; - llvm::Type *PipeROTy; - llvm::Type *PipeWOTy; + llvm::Type *PipeTy; llvm::Type *SamplerTy; /// Structure for enqueued block information. @@ -50,34 +49,31 @@ class CGOpenCLRuntime { /// Maps block expression to block information. llvm::DenseMap<const Expr *, EnqueuedBlockInfo> EnqueuedBlockMap; - virtual llvm::Type *getPipeType(const PipeType *T, StringRef Name, - llvm::Type *&PipeTy); llvm::PointerType *getPointerType(const Type *T); public: - CGOpenCLRuntime(CodeGenModule &CGM) : CGM(CGM), - PipeROTy(nullptr), PipeWOTy(nullptr), SamplerTy(nullptr) {} - virtual ~CGOpenCLRuntime(); + CGOpenCLRuntime(CodeGenModule &CGM) + : CGM(CGM), PipeTy(nullptr), SamplerTy(nullptr) {} + ~CGOpenCLRuntime(); /// Emit the IR required for a work-group-local variable declaration, and add /// an entry to CGF's LocalDeclMap for D. The base class does this using /// CodeGenFunction::EmitStaticVarDecl to emit an internal global for D. - virtual void EmitWorkGroupLocalVarDecl(CodeGenFunction &CGF, - const VarDecl &D); + void EmitWorkGroupLocalVarDecl(CodeGenFunction &CGF, const VarDecl &D); - virtual llvm::Type *convertOpenCLSpecificType(const Type *T); + llvm::Type *convertOpenCLSpecificType(const Type *T); - virtual llvm::Type *getPipeType(const PipeType *T); + llvm::Type *getPipeType(const PipeType *T); llvm::Type *getSamplerType(const Type *T); // Returns a value which indicates the size in bytes of the pipe // element. - virtual llvm::Value *getPipeElemSize(const Expr *PipeArg); + llvm::Value *getPipeElemSize(const Expr *PipeArg); // Returns a value which indicates the alignment in bytes of the pipe // element. - virtual llvm::Value *getPipeElemAlign(const Expr *PipeArg); + llvm::Value *getPipeElemAlign(const Expr *PipeArg); /// \return __generic void* type. llvm::PointerType *getGenericVoidPointerType(); _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
