[PATCH] D77792: [analyzer] Extend constraint manager to be able to compare simple SymSymExprs
Balázs Benics via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 18 00:51:31 PDT 2020
steakhal abandoned this revision.
steakhal added a comment.
I no longer think that we should support Symbol to Symbol comparisons.
It would introduce certain anomalies, like //Why could the CM reason about this and that comparisons wile could not in others//.
As of now, it's clear that we can not compare Symbols to Symbols - preventing such confusing questions to arise in the future.
If we were to implement Symbol to Symbol comparisons, we should cover a lot more cases than this patch could.
So I decided to abandon this patch.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77792/new/
https://reviews.llvm.org/D77792
More information about the cfe-commits
mailing list