[PATCH] D80991: [WIP][Attributor] AAPotentialValues Attribute
Shinji Okumura via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 17 11:51:04 PDT 2020
okura added a comment.
In D80991#2096230 <https://reviews.llvm.org/D80991#2096230>, @jdoerfert wrote:
> In D80991#2095399 <https://reviews.llvm.org/D80991#2095399>, @okura wrote:
>
> > I found why it doesn't work. In 'AAValueSimplify::updateImpl' , sometimes 'askSimplifiedValueForAAPotentialValues' is not called and the dependency between them is not recorded. So I changed it to always call 'askSimplifiedValueForAAPotentialValues'.
> > I will add logic for binary operators to 'AAPotentialValues::updateImpl'
>
>
> So you'll update this shortly, right?
Yes, I'll update this patch soon.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80991/new/
https://reviews.llvm.org/D80991
More information about the llvm-commits
mailing list