[PATCH] D143167: [PHITransAddr] Simplify casts in PHITransAddr

[Max Kazantsev via Phabricator via llvm-commits]

mkazantsev added inline comments.


================
Comment at: llvm/lib/Analysis/PHITransAddr.cpp:189
+      return addAsInput(V);
+    }
 
----------------
kachkov98 wrote:
> mkazantsev wrote:
> > 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?
> I agree, after this change it can fold zext/sext + trunc for example (didn't see any changes on test-suite though).
Can you write a test yourself?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143167