martong reopened this revision.
martong added a comment.
This revision is now accepted and ready to land.

I have changed the dump methods to order the equivalence classes and the 
disequality info available before printing them out. The ordering is done based 
on their string representation. This comes necessarily with some performance 
penalty, though otherwise it is impossible (or practically way too cumbersome) 
to write deterministic tests. Also, the dump is used either from the debugger 
or from small test cases or when we create an exploded graph (which is again 
intended to be used during debugging), so I think that the penalty induced by 
the sorting won't be an issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103967

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to