[PATCH] D22194: Power9 - Add exploitation of oris/ori fusion

Chuang-Yu Cheng via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 11 18:16:54 PDT 2016


cycheng added a comment.

In http://reviews.llvm.org/D22194#480485, @MatzeB wrote:

> Can you explain why you need a custom mutator here? At first glance it looks like overriding TargetInstrInfo::shouldScheduleAdjacent() and using the default MacroFusion mutator is enough.


No, because default MacroFusion can only detect one pattern:

  XXXInstr (on X86: Test, Cmp, Inc; On AArch64: CMN, CMP, TST, ALU)
  BranchInstr

Also I didn't see any simple way to make MacroFusion more generic, so that's why we added a customized MacroFusion.


http://reviews.llvm.org/D22194





More information about the llvm-commits mailing list