[PATCH] D85632: [Attributor] Implement AAPotentialValues and connect it with AAValueSimplify

[Shinji Okumura via Phabricator via llvm-commits]

okura added inline comments.


================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:3490
-  BooleanState IsValidState;
-
   /// Container for potential values
----------------
jdoerfert wrote:
> I failed to determine why this cannot be private anymore.
I thought `PVS.IsValidState` in operators violate access specifier but that was my mistake, sorry.
I'll fix this.


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

https://reviews.llvm.org/D85632