[PATCH] D156538: [AArch64] Try to combine FMUL with FDIV
JinGu Kang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 10 06:31:57 PDT 2023
jaykang10 added a comment.
@dmgreen As you can see, `DAGCombiner::visitFDIV` does the conversion with `Options.UnsafeFPMath || Flags.hasAllowReciprocal()` so the patterns with `fmul` are not selected without the options.
In order to use only `fmul`, we could need to remove the `Options.UnsafeFPMath || Flags.hasAllowReciprocal()`. We could need to support patterns with both `fmul` and `fdiv` with different complex patterns. How do you think about it?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156538/new/
https://reviews.llvm.org/D156538
More information about the llvm-commits
mailing list