Remove unused function from StripFuncQuantTypes

[sdasgup3]

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

[llvmbot]

@llvm/pr-subscribers-mlir

@llvm/pr-subscribers-mlir-quant

Author: Sandeep Dasgupta (sdasgup3)

Changes

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 --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 {
 

[sdasgup3]

@rafaelubalmw Can you please take a look at this?

[GleasonK]

Agree this seems unused. Will give a few days for others to see before merging.