[Mlir-commits] [mlir] a52332d - [mlir][llvm] Add FastmathFlagsInterface only once (NFC).

Tobias Gysi llvmlistbot at llvm.org
Fri Feb 24 06:57:09 PST 2023


Author: Tobias Gysi
Date: 2023-02-24T15:47:47+01:00
New Revision: a52332df5d1f9de05d9cd5b4a02b0888e28aa5cf

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

LOG: [mlir][llvm] Add FastmathFlagsInterface only once (NFC).

Previously, the FastmathFlagsInterface has been added twice to
the LLVM::PowIOp. Once directly and once indirectly using the
requiresFastmath flag. This revision only uses the flag and drops
the redundant interface.

Reviewed By: definelicht

Differential Revision: https://reviews.llvm.org/D144702

Added: 
    

Modified: 
    mlir/include/mlir/Dialect/LLVMIR/LLVMIntrinsicOps.td

Removed: 
    


################################################################################
diff  --git a/mlir/include/mlir/Dialect/LLVMIR/LLVMIntrinsicOps.td b/mlir/include/mlir/Dialect/LLVMIR/LLVMIntrinsicOps.td
index 08aca3886ba25..8ef2dc5defa35 100644
--- a/mlir/include/mlir/Dialect/LLVMIR/LLVMIntrinsicOps.td
+++ b/mlir/include/mlir/Dialect/LLVMIR/LLVMIntrinsicOps.td
@@ -109,8 +109,7 @@ def LLVM_FTruncOp : LLVM_UnaryIntrOpF<"trunc">;
 def LLVM_SqrtOp : LLVM_UnaryIntrOpF<"sqrt">;
 def LLVM_PowOp : LLVM_BinarySameArgsIntrOpF<"pow">;
 def LLVM_PowIOp : LLVM_OneResultIntrOp<"powi", [], [0,1],
-    [DeclareOpInterfaceMethods<FastmathFlagsInterface>, Pure],
-    /*requiresFastmath=*/1> {
+                                       [Pure], /*requiresFastmath=*/1> {
   let arguments =
       (ins LLVM_ScalarOrVectorOf<LLVM_AnyFloat>:$val,
            AnySignlessInteger:$power,


        


More information about the Mlir-commits mailing list