[PATCH] D60413: [BDCE] SExt -> ZExt when no sign bits is used and instruction has multiple uses
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 8 09:25:04 PDT 2020
nikic requested changes to this revision.
nikic added inline comments.
This revision now requires changes to proceed.
================
Comment at: llvm/lib/Transforms/Scalar/BDCE.cpp:127
+ Changed = true;
+ NumSExt2ZExt++;
+ continue;
----------------
You probably need to `clearAssumptionsOfUsers()` here. Please check this test case: https://alive2.llvm.org/ce/z/caMis2
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60413/new/
https://reviews.llvm.org/D60413
More information about the llvm-commits
mailing list