[llvm] Introduce mcdc::TVIdxBuilder (LLVM side, NFC) (PR #80676)

[via llvm-commits]

ZhuUx wrote:

Sure, I’d collate it. Please wait some time.





来自钉钉专属商务邮箱------------------------------------------------------------------
发件人：NAKAMURA ***@***.***>
日　期：2024年06月26日 19:03:11
***@***.***>
***@***.***>; ***@***.***>
主　题：Re: [llvm/llvm-project] Introduce mcdc::TVIdxBuilder (LLVM side, NFC) (PR #80676)

@chapuni commented on this pull request. 
In llvm/lib/ProfileData/Coverage/CoverageMapping.cpp:
> +  // Construct Nodes and set up each InCount
+  auto N = NextIDs.size();
+  SmallVector<MCDCNode> Nodes(N);
+  for (unsigned ID = 0; ID < N; ++ID) {
+    for (unsigned C = 0; C < 2; ++C) {
+#ifndef NDEBUG
+      Indices[ID][C] = INT_MIN;
+#endif
+      auto NextID = NextIDs[ID][C];
+      Nodes[ID].NextIDs[C] = NextID;
+      if (NextID >= 0)
+        ++Nodes[NextID].InCount;
+    }
+  }
+
+  // Sort key ordered by <-Width, Ord>

Interesting. Could you introduce me the discussion, or paste the problematic cond tree anywhere?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: ***@***.***> 


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