[PATCH] D109700: [InstCombine] Improve TryToSinkInstruction with multiple uses
Anna Thomas via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 13 09:03:43 PDT 2021
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>
More information about the llvm-commits
mailing list