[PATCH] D136442: [GISel] Add (fsub +-0.0, X) -> fneg combine

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 08:03:14 PDT 2022


arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp:5251
+  const auto LHSCst = getFConstantVRegValWithLookThrough(LHS, MRI);
+  return LHSCst && LHSCst->Value.isZero();
+}
----------------
arsenm wrote:
> Doing this for -0 is only valid with nsz: [[ https://alive2.llvm.org/ce/z/SK2Bs9 ]]
I mean +0 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136442/new/

https://reviews.llvm.org/D136442



More information about the llvm-commits mailing list