[PATCH] D80175: [PowerPC][MachineCombiner] reassociate fma to expose more ILP

Mon Jun 8 23:57:47 PDT 2020

shchenz added a comment.

In D80175#2081483 <https://reviews.llvm.org/D80175#2081483>, @AaronLiu wrote:

> Can you add a hidden option with init false?  You can turn it true later on.
>  So that people can try with your option off and on? Thanks!

Have you met some issues with this being turned on? This is specific for PowerPC, I have verified that on PowerPC there is no deg for the patch on the benchmarks I run. If you just want to see the impact of this patch, I would suggest you comment out the two newly added lines in function `PPCInstrInfo::getMachineCombinerPatterns`, it will turn off this opt.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80175