[all-commits] [llvm/llvm-project] 8253a8: [InstCombine] Erase old mul when creating umulo

[Nikita Popov via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 8253a86b65c626f470e90ddacba1166b81012b12
      https://github.com/llvm/llvm-project/commit/8253a86b65c626f470e90ddacba1166b81012b12
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-03-29 (Sun, 29 Mar 2020)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp

  Log Message:
  -----------
  [InstCombine] Erase old mul when creating umulo

As we don't return the result of replaceInstUsesWith(), we are
responsible for erasing the instruction.

There is a small subtlety here in that we need to do this after
the other uses of Builder, which uses the original multiply as
the insertion point.

NFC apart from worklist order changes.