[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