[all-commits] [llvm/llvm-project] f50268: [MergeICmps] Relax sinking check

[Nikita Popov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f502683750f4240c48a3e24bf023294e4a27929e
      https://github.com/llvm/llvm-project/commit/f502683750f4240c48a3e24bf023294e4a27929e
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2021-07-23 (Fri, 23 Jul 2021)

  Changed paths:
    M llvm/lib/Transforms/Scalar/MergeICmps.cpp
    M llvm/test/Transforms/MergeICmps/X86/split-block-does-work.ll

  Log Message:
  -----------
  [MergeICmps] Relax sinking check

The check for sinking instructions past the load + cmp sequence
currently checks for side-effects, which includes writing to memory
and unwinding. However, I don't believe we care about sinking the
instructions past an unwind (as they don't have any side-effects
themselves).

Differential Revision: https://reviews.llvm.org/D106591