[clang] [clang][analyzer] Stable order for SymbolRef-keyed containers (PR #121551)

Balazs Benics via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 3 01:47:56 PST 2025


================
@@ -56,6 +68,8 @@ class SymExpr : public llvm::FoldingSetNode {
 
   Kind getKind() const { return K; }
 
+  SymbolID getSymbolID() const { return Sym; }
----------------
steakhal wrote:

This is expected to be a fairly infrequently used API. I think this deserves to be highlighted on this getter. Especially because modern IDEs will only display these notes and not the notes at the `Sym` declaration.

In other words, why would / where would anyone use this API?
In this patch I could only see a single caller, `ImutContainerInfo`.

https://github.com/llvm/llvm-project/pull/121551


More information about the cfe-commits mailing list