[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