Skip to content

Conversation

@kazutakahirata
Copy link
Contributor

This patch removes getDestAlignment and getSourceAlignment as they
have been deprecated for more than two years since:

commit 135f23d
Author: Guillaume Chatelet [email protected]
Date: Mon Jan 16 12:34:40 2023 +0000

I'm not aware of any downstream users AFAIK.

This patch removes getDestAlignment and getSourceAlignment as they
have been deprecated for more than two years since:

  commit 135f23d
  Author: Guillaume Chatelet <[email protected]>
  Date:   Mon Jan 16 12:34:40 2023 +0000

I'm not aware of any downstream users AFAIK.
@llvmbot
Copy link
Member

llvmbot commented Jul 30, 2025

@llvm/pr-subscribers-llvm-ir

Author: Kazu Hirata (kazutakahirata)

Changes

This patch removes getDestAlignment and getSourceAlignment as they
have been deprecated for more than two years since:

commit 135f23d
Author: Guillaume Chatelet <[email protected]>
Date: Mon Jan 16 12:34:40 2023 +0000

I'm not aware of any downstream users AFAIK.


Full diff: https://github.com/llvm/llvm-project/pull/151282.diff

1 Files Affected:

  • (modified) llvm/include/llvm/IR/IntrinsicInst.h (-17)
diff --git a/llvm/include/llvm/IR/IntrinsicInst.h b/llvm/include/llvm/IR/IntrinsicInst.h
index 0318427bf8a1c..2e1389633d7e4 100644
--- a/llvm/include/llvm/IR/IntrinsicInst.h
+++ b/llvm/include/llvm/IR/IntrinsicInst.h
@@ -996,14 +996,6 @@ template <typename Derived> class MemIntrinsicBase : public IntrinsicInst {
     return cast<PointerType>(getRawDest()->getType())->getAddressSpace();
   }
 
-  /// FIXME: Remove this function once transition to Align is over.
-  /// Use getDestAlign() instead.
-  LLVM_DEPRECATED("Use getDestAlign() instead", "getDestAlign")
-  unsigned getDestAlignment() const {
-    if (auto MA = getParamAlign(ARG_DEST))
-      return MA->value();
-    return 0;
-  }
   MaybeAlign getDestAlign() const { return getParamAlign(ARG_DEST); }
 
   /// Set the specified arguments of the instruction.
@@ -1057,15 +1049,6 @@ template <class BaseCL> class MemTransferBase : public BaseCL {
     return cast<PointerType>(getRawSource()->getType())->getAddressSpace();
   }
 
-  /// FIXME: Remove this function once transition to Align is over.
-  /// Use getSourceAlign() instead.
-  LLVM_DEPRECATED("Use getSourceAlign() instead", "getSourceAlign")
-  unsigned getSourceAlignment() const {
-    if (auto MA = BaseCL::getParamAlign(ARG_SOURCE))
-      return MA->value();
-    return 0;
-  }
-
   MaybeAlign getSourceAlign() const {
     return BaseCL::getParamAlign(ARG_SOURCE);
   }

@kazutakahirata kazutakahirata merged commit 2920c43 into llvm:main Jul 30, 2025
11 checks passed
@kazutakahirata kazutakahirata deleted the cleanup_20250729_remove_deprecated_llvm_IR_getDestAlignment branch July 30, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants