[PATCH] D112721: [SCCP] Tune `zext` and `sext` processing for overdefined operand

[Anton Afanasyev via Phabricator via llvm-commits]

anton-afanasyev marked an inline comment as done.
anton-afanasyev added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/SCCPSolver.cpp:837
+                                      I.getOpcode() == Instruction::SExt)) {
+    ConstantRange OpRange = ConstantRange::getFull(
+        I.getOperand(0)->getType()->getScalarSizeInBits());
----------------
fhahn wrote:
> could that be handled by the code above, just with using the full range for OpRange if it is overdefined?
Ok, changed it this way, though getting a few more redundant computations.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112721/new/

https://reviews.llvm.org/D112721