[PHITransAddr] Simplify casts in PHITransAddr

[kachkov98]

Summary

Try to simplify cast in similar way as for GEP and ADD with constant (e.g. sext/zext + trunc).

[mkazantsev]

llvm/lib/Analysis/PHITransAddr.cpp

189

I'm fine with isSafeToSpeculativelyExecute part, it seems reasonable. But this doesn't seem to be NFC while whole patch claims it's an NFC. Should it be 2 different patches?

[kachkov98]

llvm/lib/Analysis/PHITransAddr.cpp

189

I agree, after this change it can fold zext/sext + trunc for example (didn't see any changes on test-suite though).

[kachkov98]

Split patch

[mkazantsev]

llvm/lib/Analysis/PHITransAddr.cpp

189

Can you write a test yourself?

[mkazantsev]

Please add a test. If you can't figure it out yourself, you can insert an assert and run a big enough tests corps to see if something changed, and then use bugpoint to reduce it. Good luck!

[kachkov98]

Added synthetic test for simplifyCastInst

[kachkov98]

Separate patch for removing isSafeToSpeculativelyExecute() checks: https://reviews.llvm.org/D143255

[mkazantsev]

Looks reasonable. I'm not familiar with this code part in general, so please wait some time in case if someone has concerns, but looks pretty straightforward.

[kachkov98]

In D143167#4102182, @mkazantsev wrote:

Looks reasonable. I'm not familiar with this code part in general, so please wait some time in case if someone has concerns, but looks pretty straightforward.

Ok, thank you for the reviews!

llvm/lib/Analysis/PHITransAddr.cpp

Value *PHITransAddr::translateSubExpr(Value *V, BasicBlock *CurBB,
   if (CastInst *Cast = dyn_cast<CastInst>(Inst)) {
     Value *PHIIn = translateSubExpr(Cast->getOperand(0), CurBB, PredBB, DT);
     if (!PHIIn) return nullptr;
     if (PHIIn == Cast->getOperand(0))
       return Cast;
 
     // Find an available version of this cast.
 
-    // Constants are trivial to find.
-    if (Constant *C = dyn_cast<Constant>(PHIIn))
-      return addAsInput(
-          ConstantExpr::getCast(Cast->getOpcode(), C, Cast->getType()));
+    // Try to simplify cast first.
+    if (Value *V = simplifyCastInst(Cast->getOpcode(), PHIIn, Cast->getType(),
+                                    {DL, TLI, DT, AC})) {
+      RemoveInstInputs(PHIIn, InstInputs);
+      return addAsInput(V);
+    }
 
     // Otherwise we have to see if a casted version of the incoming pointer
     // is available.  If so, we can use it, otherwise we have to fail.
     for (User *U : PHIIn->users()) {
       if (CastInst *CastI = dyn_cast<CastInst>(U))
         if (CastI->getOpcode() == Cast->getOpcode() &&
             CastI->getType() == Cast->getType() &&
             (!DT || DT->dominates(CastI->getParent(), PredBB)))