[PATCH] D85632: [Attributor] Implement AAPotentialValues and connect it with AAValueSimplify
Shinji Okumura via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 10 10:29:14 PDT 2020
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
More information about the llvm-commits
mailing list