[llvm] [SystemZ] Enable MachineCombiner for FP reassociation (PR #83546)

Ulrich Weigand via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 1 09:01:11 PST 2024


uweigand wrote:

> A bit surprisingly, I saw instead a 13% improvement imagick _without_ the MachineCombiner, just because of the global reg/mem folding I tried. 

Haven't looked into the patch in detail yet, but this is interesting.  Should we then always do the reg/mem folding late (for all FP operations, including non-reassociable ones), if it is generally beneficial?  This might actually even simplify the code (and could be done as a separate PR before the MachineCombiner reassociation patch?).

https://github.com/llvm/llvm-project/pull/83546


More information about the llvm-commits mailing list