[Mlir-commits] [mlir] [TOSA] Usage of 32bit integer for 'index to float' in rfft2d (PR #75098)

Dmitriy Smirnov llvmlistbot at llvm.org
Wed Dec 13 04:29:41 PST 2023


================
@@ -2284,8 +2284,11 @@ struct RFFT2dConverter final : public OpRewritePattern<RFFT2dOp> {
 
   static Value castIndexToFloat(OpBuilder &builder, Location loc,
                                 FloatType type, Value value) {
-    auto integerVal =
-        builder.create<arith::IndexCastUIOp>(loc, builder.getI64Type(), value);
+    auto integerVal = builder.create<arith::IndexCastUIOp>(
+        loc,
+        32 < type.getIntOrFloatBitWidth() ? builder.getI64Type()
----------------
d-smirnov wrote:

tosa.rfft2d defined only for f32, probably we should not bother unless there is some strong case for this. 

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


More information about the Mlir-commits mailing list