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

NAKAMURA Takumi via cfe-commits cfe-commits at lists.llvm.org
Wed May 22 14:17:08 PDT 2024


================
@@ -339,8 +365,18 @@ class CoverageMappingBuilder {
 
     llvm::SmallSet<FileID, 8> Visited;
     SmallVector<std::pair<SourceLocation, unsigned>, 8> FileLocs;
-    for (const auto &Region : SourceRegions) {
+    for (auto &Region : SourceRegions) {
       SourceLocation Loc = Region.getBeginLoc();
+
+      // Replace Region with its definition if it is in <scratch space>.
----------------
chapuni wrote:

Now this collides to #91446 . Could you update?
I suggest this may come in advance of #91446.

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


More information about the cfe-commits mailing list