[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

185

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

185

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

185

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!