[PATCH] D109700: [InstCombine] Improve TryToSinkInstruction with multiple uses

[Anna Thomas via Phabricator via llvm-commits]

anna created this revision.
anna added reviewers: jdoerfert, Tyker, mkazantsev, nikic.
Herald added subscribers: dexonsmith, uenoku, hiraditya.
Herald added a reviewer: uenoku.
anna requested review of this revision.
Herald added a reviewer: sstefan1.
Herald added a reviewer: baziotis.
Herald added a project: LLVM.

We were restricting the number of undroppable uses to one for sinking an
instruction to user block. However, we can have multiple uses in a
single user.
The API has been updated accordingly since in both usecases (Attributor
and InstCombine), we seem to care about the user.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D109700

Files:
  llvm/include/llvm/IR/Value.h
  llvm/lib/IR/Value.cpp
  llvm/lib/Transforms/InstCombine/InstructionCombining.cpp
  llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp
  llvm/test/Transforms/InstCombine/README.txt
  llvm/test/Transforms/InstCombine/icmp-mul-zext.ll
  llvm/test/Transforms/InstCombine/sink_instruction.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109700.372272.patch
Type: text/x-patch
Size: 7654 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210913/632e2666/attachment.bin>