fixes semi-regression; discard check now skips nkHiddenSubConv (#23840

) follow up #23681 ref https://forum.nim-lang.org/t/11987
nim-lang · Jul 16, 2024 · 648f82c · 648f82c
1 parent b7a275d
commit 648f82c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/compiler/semstmts.nim b/compiler/semstmts.nim
@@ -135,7 +135,7 @@ const
   skipForDiscardable = {nkStmtList, nkStmtListExpr,
     nkOfBranch, nkElse, nkFinally, nkExceptBranch,
     nkElifBranch, nkElifExpr, nkElseExpr, nkBlockStmt, nkBlockExpr,
-    nkHiddenStdConv, nkHiddenDeref}
+    nkHiddenStdConv, nkHiddenSubConv, nkHiddenDeref}
 
 proc implicitlyDiscardable(n: PNode): bool =
   # same traversal as endsInNoReturn

diff --git a/tests/discard/tdiscardable.nim b/tests/discard/tdiscardable.nim
@@ -164,3 +164,12 @@ block: # bug #23775
       let _ = retString()
 
   doStuff(C)
+
+block:
+  proc test(): (int, int) {.discardable.} =
+    discard
+
+  if true:
+    test()
+  else:
+    quit()