[PATCH] D68626: [Attributor] Use undef for calls with unused arguments.

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 12 11:04:12 PDT 2019


jdoerfert added a comment.

In D68626#1707223 <https://reviews.llvm.org/D68626#1707223>, @sstefan1 wrote:

> I have no problem with this going in as is. I have one question though. Why do we have to wait for ValueSimplify to finish? Wouldn't it be useful for AAIsDead to have `isDeadArg(Arg)` and then ValueSimplify could use that to decide whether to simplify or not?


You are right. I'll replace it with a `AAIsDead` solution instead shortly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68626





More information about the llvm-commits mailing list