[mlir][llvm] Add FastmathFlagsInterface only once (NFC).
authorTobias Gysi <tobias.gysi@nextsilicon.com>
Fri, 24 Feb 2023 14:47:23 +0000 (15:47 +0100)
committerTobias Gysi <tobias.gysi@nextsilicon.com>
Fri, 24 Feb 2023 14:47:47 +0000 (15:47 +0100)
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

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

index 08aca38..8ef2dc5 100644 (file)
@@ -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,