[PATCH] D75815: [InstCombine] Simplify calls with "returned" attribute

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp:4569
 
+  if (!Call.use_empty() && !Call.isMustTailCall())
+    if (Value *ReturnedArg = Call.getReturnedArgOperand())
----------------
nickdesaulniers wrote:
> re: https://bugs.llvm.org/show_bug.cgi?id=46463
> 
> Do we need to check that the callee doesn't have the `noinline` function attribute?
I'm not sure i understand why presence of `noinline` should block this?
`noipa` on the other hand, sure.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75815