[llvm] [llvm-cov][CoverageView] minor fix/improvement to HTML and text coverage output (PR #80952)

Alan Phipps via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 7 06:21:01 PST 2024


================
@@ -990,15 +993,13 @@ void SourceCoverageViewHTML::renderMCDCView(raw_ostream &OS, MCDCView &MRV,
     std::string ColNoStr = Twine(DecisionRegion.ColumnStart).str();
     std::string TargetName = "L" + LineNoStr;
     OS << tag("span",
-              a("#" + TargetName, tag("span", LineNoStr + ":" + ColNoStr),
-                TargetName),
+              a("#" + TargetName, tag("span", LineNoStr + ":" + ColNoStr)),
----------------
evodius96 wrote:

What was the problem with generating the "a name=" for the link? The same is done for the Branch coverage view in the routine above this.  Was it just unnecessary?

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


More information about the llvm-commits mailing list