[PATCH] D77977: [InstCombine] Simplify calls with casted "returned" attribute

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp:4660
+      Type *RetArgTy = ReturnedArg->getType();
+      if (RetArgTy->canLosslesslyBitCastTo(CallTy))
+        return replaceInstUsesWith(
----------------
nikic wrote:
> lebedev.ri wrote:
> > I can't imagine it matters, but these should be swapped around.
> Shouldn't this be an assertion rather than an if check?
Is there a verifier check for that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77977