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

Mon Mar 23 07:03:34 PDT 2020

spatel added inline comments.

================
Comment at: llvm/test/CodeGen/PowerPC/fma-negate.ll:289
+; NO-VSX-NEXT:    fmadd 1, 1, 3, 5
+; NO-VSX-NEXT:    fmadd 2, 0, 4, 6
 ; NO-VSX-NEXT:    blr
----------------
qiucf wrote:
> This should be a regression. And the reason is `SplitVectorRes` did not pass `SDNode` flags to legalized ones. If that's truly a bug, I will cook a patch for it.
This should **not** be a regression, right? If we dropped the flags during legalization, that is a bug.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76585