[PATCH] D85639: [Attributor] Fix bug in operator of PotentialValuesState

Shinji Okumura via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 10 05:44:58 PDT 2020


okura added a comment.

In D85639#2206325 <https://reviews.llvm.org/D85639#2206325>, @lebedev.ri wrote:

> Test?

In the current situation of AAPotentialValues, I think it is difficult to reflect this change in tests. I noticed this bug when I worked on D85632 <https://reviews.llvm.org/D85632> and saw affected tests because `AAPotentialValues` has interface only.
Should I include the changes here in that patch? (But I feel that the patch is big and want to split it if possible.)


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

https://reviews.llvm.org/D85639



More information about the llvm-commits mailing list