[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