[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