[Mlir-commits] [mlir] [MLIR][LLVMIR] Add an op for the llvm.modf intrinsic (PR #223224)

[Letu Ren]

FantasqueX wrote:

> There is already an open PR that adds modf:
> 
> #222355
> 
> It has not been updated to print the fastmath flags the same way as LLVM_SincosOp does after the recent changes. @FantasqueX would you want to update your PR to match the new assembly format (then we could land yours)? Alternatively, we could also update the verifier of this PR to have a common helper for the verifier.

OK. I would update my PR to include fastmath flags. It will happen in 1~2 days, I think.

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