[Mlir-commits] [mlir] Remove unused function from StripFuncQuantTypes (PR #121594)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Fri Jan 3 11:46:15 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir-quant

Author: Sandeep Dasgupta (sdasgup3)

<details>
<summary>Changes</summary>

`StripFuncQuantTypes::isLegalType` is  unused and can be safely removed. 

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


1 Files Affected:

- (modified) mlir/lib/Dialect/Quant/Transforms/StripFuncQuantTypes.cpp (-8) 


``````````diff
diff --git a/mlir/lib/Dialect/Quant/Transforms/StripFuncQuantTypes.cpp b/mlir/lib/Dialect/Quant/Transforms/StripFuncQuantTypes.cpp
index 71b88d1be1b05b..02384de27ab5e6 100644
--- a/mlir/lib/Dialect/Quant/Transforms/StripFuncQuantTypes.cpp
+++ b/mlir/lib/Dialect/Quant/Transforms/StripFuncQuantTypes.cpp
@@ -65,14 +65,6 @@ class QuantizedTypeConverter : public TypeConverter {
 class StripFuncQuantTypes
     : public impl::StripFuncQuantTypesBase<StripFuncQuantTypes> {
 
-  // Return whether a type is considered legal when occurring in the header of
-  // a function or as an operand to a 'return' op.
-  static bool isLegalType(Type type) {
-    if (auto tensorType = dyn_cast<TensorType>(type))
-      return isLegalType(tensorType.getElementType());
-    return !isa<quant::QuantizedType>(type);
-  }
-
 public:
   void runOnOperation() override {
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/121594


More information about the Mlir-commits mailing list