[PATCH] D72392: [MachineScheduler] Fix the TopDepth/BotHeightReduce latency heuristics

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 9 04:04:46 PST 2020


foad added a comment.

The diff in `llvm/test/CodeGen/X86/testb-je-fusion.ll` is:

          movl    %edi, %eax
  -       addl    $-512, %eax             # imm = 0xFE00
          movb    $1, (%rsi)
  +       addl    $-512, %eax             # imm = 0xFE00
          je      .LBB2_2
  ...
          movl    %edi, %eax
  -       decl    %eax
          movb    $1, (%rsi)
  +       decl    %eax
          je      .LBB3_2

The scheduler prefers not to put the addl/decl immediately after the first movl because of the register dependency on eax with latency 1.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72392





More information about the llvm-commits mailing list