[clang] [analyzer][CTU] Macro expansions for imported translation units (PR #176126)

[Balázs Benics via cfe-commits]

================
@@ -186,14 +186,14 @@ class CrossTranslationUnitContext {
   /// Emit diagnostics for the user for potential configuration errors.
   void emitCrossTUDiagnostics(const IndexError &IE);
 
-  /// Returns the MacroExpansionContext for the imported TU to which the given
-  /// source-location corresponds.
-  /// \p ToLoc Source location in the imported-to AST.
-  /// \note If any error happens such as \p ToLoc is a non-imported
+  /// Returns the MacroExpansionContext for the imported TU and the location in
+  /// the imported-from AST to which the given source-location corresponds. \p
+  /// ToLoc Source location in the imported-to AST. \note If any error happens
+  /// such as \p ToLoc is a non-imported
   ///       source-location, empty is returned.
   /// \note Macro expansion tracking for imported TUs is not implemented yet.
   ///       It returns empty unconditionally.
-  std::optional<clang::MacroExpansionContext>
+  std::optional<std::pair<MacroExpansionContext, SourceLocation>>
----------------
steakhal wrote:

Why do we need a pair here now?

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