[PATCH] D110943: [GlobalISel] Combine fabs(fneg(x)) to fabs(x)
Jay Foad via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 1 08:37:48 PDT 2021
foad added inline comments.
================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/CombinerHelper.h:371-376
/// Match fabs(fabs(x)) to fabs(x).
bool matchCombineFAbsOfFAbs(MachineInstr &MI, Register &Src);
void applyCombineFAbsOfFAbs(MachineInstr &MI, Register &Src);
+ /// Transform fabs(fneg(x)) to fabs(x).
+ bool matchCombineFAbsOfFNeg(MachineInstr &MI, BuildFnTy &MatchInfo);
----------------
Would it be neater to combine these into a single "fabs of fabs-or-fneg" combine?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110943/new/
https://reviews.llvm.org/D110943
More information about the llvm-commits
mailing list