[PATCH] D103967: [Analyzer][solver] Add dump methods for (dis)equality classes.
Valeriy Savchenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jun 28 07:05:10 PDT 2021
vsavchenko added a comment.
In D103967#2844140 <https://reviews.llvm.org/D103967#2844140>, @thakis wrote:
> Thanks. Looks like it flakily fails on mac too every now and then: http://45.33.8.238/mac/33005/step_7.txt
>
> Maybe you print something in nondeterministic iteration order?
Yes, I think that the problem is deeper. `ClassMembers` is `llvm::ImmutableSet` (i.e. sorted set), and it uses `SymbolRef` (e.g. pointer values) for sorting. So, it does look like the order can be nondeterministic.
However, when we print constraints, we should see similar problems, but we don't 🤔
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103967/new/
https://reviews.llvm.org/D103967
More information about the cfe-commits
mailing list