From bc9ae8418aa566d9f1c52250b919a8463288a141 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicolai=20H=C3=A4hnle?= Date: Mon, 12 Jan 2026 09:42:05 -0800 Subject: [PATCH] [CodeGen] Remove legacy getTgtMemIntrinsic overload It is now fully unused. commit-id:028dd72d --- llvm/include/llvm/CodeGen/TargetLowering.h | 20 +------------------- 1 file changed, 1 insertion(+), 19 deletions(-) diff --git a/llvm/include/llvm/CodeGen/TargetLowering.h b/llvm/include/llvm/CodeGen/TargetLowering.h index 37002d3bc227f..525e84dfe2acb 100644 --- a/llvm/include/llvm/CodeGen/TargetLowering.h +++ b/llvm/include/llvm/CodeGen/TargetLowering.h @@ -1250,26 +1250,8 @@ class LLVM_ABI TargetLoweringBase { /// access multiple memory locations. virtual void getTgtMemIntrinsic(SmallVectorImpl &Infos, const CallBase &I, MachineFunction &MF, - unsigned Intrinsic) const { - // The default implementation forwards to the legacy single-info overload - // for compatibility. - IntrinsicInfo Info; - if (getTgtMemIntrinsic(Info, I, MF, Intrinsic)) - Infos.push_back(Info); - } + unsigned Intrinsic) const {} -protected: - /// This is a legacy single-info overload. New code should override the - /// SmallVectorImpl overload instead to support multiple memory operands. - /// - /// TODO: Remove this once the refactoring is complete. - virtual bool getTgtMemIntrinsic(IntrinsicInfo &, const CallBase &, - MachineFunction &, - unsigned /*Intrinsic*/) const { - return false; - } - -public: /// Returns true if the target can instruction select the specified FP /// immediate natively. If false, the legalizer will materialize the FP /// immediate as a load from a constant pool.