[PATCH] D125321: [InstCombine] Freeze other uses of frozen value
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 11 03:02:20 PDT 2022
nikic updated this revision to Diff 428605.
nikic added a comment.
Insert the freeze directly after the definition of the op, rather than at the nearest common dominator of the uses.
Finding the correct insert position "directly after the def" is annoyingly tricky...
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125321/new/
https://reviews.llvm.org/D125321
Files:
llvm/lib/Transforms/InstCombine/InstCombineInternal.h
llvm/lib/Transforms/InstCombine/InstructionCombining.cpp
llvm/test/Transforms/InstCombine/freeze.ll
llvm/test/Transforms/InstCombine/onehot_merge.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D125321.428605.patch
Type: text/x-patch
Size: 10480 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220511/d135640f/attachment-0001.bin>
More information about the llvm-commits
mailing list