[PATCH] D125746: [PowerPC] Treat llvm.fmuladd intrinsic as using CTR

[ChenZheng via Phabricator via llvm-commits]

shchenz added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp:505
+              return true;
+            else
+              LLVM_FALLTHROUGH;
----------------
no need for `else`. 
And why we need `LLVM_FALLTHROUGH`, I don't see any handling for `Intrinsic::fmuladd` in the switch block below, will `break` work?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125746