[clang] [Coverage][Expansion] handle nested macros in scratch space (PR #89869)

[Wentao Zhang via cfe-commits]

================
@@ -517,7 +552,7 @@ class CoverageMappingBuilder {
     SourceRegionFilter Filter;
     for (const auto &FM : FileIDMapping) {
       SourceLocation ExpandedLoc = FM.second.second;
-      SourceLocation ParentLoc = getIncludeOrExpansionLoc(ExpandedLoc);
+      SourceLocation ParentLoc = getIncludeOrExpansionLoc(ExpandedLoc, false);
----------------
whentojump wrote:

Hi, in the initial version we were actually using a separate function (but with 
some duplicate code). See this commit 
https://github.com/llvm/llvm-project/pull/89869/commits/145c807a177db8e649f7f3343d640a000e65cebb

Do you suggest that way is better?

https://github.com/llvm/llvm-project/pull/89869
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits