[PATCH] D91343: [InstCombine] Optimize away the unnecessary multi-use sign-extend
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 16 08:00:02 PST 2020
lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp:919-921
+ // amount, this is probably a zero/sign extension, which may be unnecessary,
+ // if the demanded bits are already within original operand. So, return the
+ // original operand instead.
----------------
s/if the demanded bits are already within original operand/if we do not demand any of the new sign bits/
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91343/new/
https://reviews.llvm.org/D91343
More information about the llvm-commits
mailing list