[PATCH] D76585: [PowerPC] Require NSZ flag for c-a*b to FNMSUB

[Qing Shan Zhang via Phabricator via llvm-commits]

steven.zhang added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:15780
+
+    if (NegN0 && N0Cost <= N1Cost) {
+      Cost = std::min(N2Cost, std::min(N0Cost, N1Cost));
----------------
As we already know that, N0Cost <= N1Cost, so, what we need to do is as:
Cost = std::min(N0Cost, N2Cost);
I will update the logic in DAGCombine there as it has similar issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76585