[PATCH] D108284: [DAGCombiner] Combine frem into fdiv+ftrunc+fma
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 8 01:59:23 PDT 2023
RKSimon added inline comments.
================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:16685
+ Flags.hasNoInfs() && !TLI.isOperationLegalOrCustom(ISD::FREM, VT) &&
+ TLI.isOperationLegalOrCustom(ISD::FTRUNC, VT)) {
+ SDLoc Loc(N);
----------------
Should we check for isOperationLegalOrCustom(ISD::FMA, VT) as well?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108284/new/
https://reviews.llvm.org/D108284
More information about the llvm-commits
mailing list