[Mlir-commits] [mlir] d81a843 - [mlir] Put back virtual ~ConversionTarget(), some users started relying on it

Benjamin Kramer llvmlistbot at llvm.org
Thu Jul 29 02:16:36 PDT 2021


Author: Benjamin Kramer
Date: 2021-07-29T11:10:07+02:00
New Revision: d81a843846f4171ffe3f44b66bf3a9fda5c411cf

URL: https://github.com/llvm/llvm-project/commit/d81a843846f4171ffe3f44b66bf3a9fda5c411cf
DIFF: https://github.com/llvm/llvm-project/commit/d81a843846f4171ffe3f44b66bf3a9fda5c411cf.diff

LOG: [mlir] Put back virtual ~ConversionTarget(), some users started relying on it

Added: 
    

Modified: 
    mlir/include/mlir/Transforms/DialectConversion.h

Removed: 
    


################################################################################
diff  --git a/mlir/include/mlir/Transforms/DialectConversion.h b/mlir/include/mlir/Transforms/DialectConversion.h
index c9e9f6748399..7354b55ecc72 100644
--- a/mlir/include/mlir/Transforms/DialectConversion.h
+++ b/mlir/include/mlir/Transforms/DialectConversion.h
@@ -622,6 +622,7 @@ class ConversionTarget {
   using DynamicLegalityCallbackFn = std::function<bool(Operation *)>;
 
   ConversionTarget(MLIRContext &ctx) : ctx(ctx) {}
+  virtual ~ConversionTarget() = default;
 
   //===--------------------------------------------------------------------===//
   // Legality Registration


        


More information about the Mlir-commits mailing list