[clang] [Lifetime Safety] Highlight lifetimebound calls in alias chain diagnostics (PR #206337)

[via cfe-commits]

================
@@ -524,13 +525,20 @@ class LifetimeChecker {
   /// Given a chain of origins that shows how a loan propagates, this function
   /// extracts the corresponding expressions for each origin. Origins that refer
   /// to declarations (rather than expressions) are skipped.
-  llvm::SmallVector<const Expr *>
-  getExprChain(llvm::ArrayRef<OriginID> OriginFlowChain) {
-    llvm::SmallVector<const Expr *> rs;
-    for (const OriginID CurrOID : OriginFlowChain)
-      if (const Expr *CurrExpr =
-              FactMgr.getOriginMgr().getOrigin(CurrOID).getExpr())
-        rs.push_back(CurrExpr);
+  llvm::SmallVector<LifetimeSafetyAliasChainEntry>
+  getAliasChain(llvm::ArrayRef<const OriginFlowFact *> OriginFlowChain) {
+    llvm::SmallVector<LifetimeSafetyAliasChainEntry> rs;
----------------
NeKon69 wrote:

Naming seems a bit too vague. Is `rs` result ?

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