[clang] [LifetimeSafety] Introduce buildOriginFlowChain for use-after-scope (PR #199345)

[Utkarsh Saxena via cfe-commits]

================
@@ -471,13 +474,50 @@ class LifetimeSafetySemaHelperImpl : public LifetimeSafetySemaHelper {
   }
 
   std::string getDiagSubjectDescription(const Expr *E) {
+    // FIXME: Ideally, this should use IgnoreParenImpCasts().
+    // However, according to the comment on IgnoreParenImpCasts(),
+    // it is not fully equivalent to IgnoreImpCasts() + IgnoreParens().
+    // Once the FIXME in IgnoreParenImpCasts() is resolved,
+    // this can be switched to use IgnoreParenImpCasts().
+    E = E->IgnoreImpCasts()->IgnoreParens();
     if (isa<MaterializeTemporaryExpr>(E))
       return "local temporary object";
 
     if (const auto *DRE = dyn_cast<DeclRefExpr>(E))
       return getDiagSubjectDescription(DRE->getDecl());
     // TODO: Handle other expression types.
-    return "";
+    return "expression";
+  }
+
+  bool shouldShowInAliasChain(const Expr *CurrExpr, const Expr *LastExpr) {
+    CurrExpr = CurrExpr->IgnoreImpCasts()->IgnoreParens();
+    LastExpr = LastExpr->IgnoreImpCasts()->IgnoreParens();
+
+    if (CurrExpr->getSourceRange() == LastExpr->getSourceRange())
+      return false;
+    if (isa<MaterializeTemporaryExpr>(CurrExpr) &&
+        isa<MaterializeTemporaryExpr>(LastExpr))
----------------
usx95 wrote:

Why do we need `isa<MaterializeTemporaryExpr>(LastExpr)` here ? Why not just `CurrExpr` ?


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