[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