Skip to content

[CodeGen] Remove legacy getTgtMemIntrinsic overload #175846

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
nhaehnle merged 1 commit into from
Feb 23, 2026
Merged

[CodeGen] Remove legacy getTgtMemIntrinsic overload #175846

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 1 addition & 19 deletions llvm/include/llvm/CodeGen/TargetLowering.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1250,26 +1250,8 @@ class LLVM_ABI TargetLoweringBase {
/// access multiple memory locations.
virtual void getTgtMemIntrinsic(SmallVectorImpl<IntrinsicInfo> &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.
Expand Down
Loading