Explain the issue and test in more detail

Clyybber · Mar 12, 2024 · ae58d49 · ae58d49
1 parent 6a6c0a3
commit ae58d49
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/tests/lang_callable/macros/tmacros_various.nim b/tests/lang_callable/macros/tmacros_various.nim
@@ -331,7 +331,12 @@ block: # bug #15118
   block:
     flop("b")
 
-block: # Ensure nkCommentStmt equality is not ignored when checking for const duplicates:
+block:
+  # Ensure nkCommentStmt equality is not ignored when vmgen.cmpNodeCnst
+  # is used to deduplicate NimNode constants, so that `CommentStmt "comment 2"`
+  # is not counted as a duplicate of `CommentStmt "comment 1"` and
+  # incorrectly optimized to point at the `Comment "comment 1"` node
+
   proc createComment(s: string): NimNode =
     result = nnkCommentStmt.newNimNode()
     result.strVal = s