Skip to content

Commit 9877b46

Browse files
committed
Update test output and remove CR-soon.
1 parent 9d530f3 commit 9877b46

File tree

2 files changed

+0
-16
lines changed

2 files changed

+0
-16
lines changed

tests/backend/checkmach/test_all_opt.ml

-10
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,3 @@ let[@zero_alloc opt] foo x = (x,x)
77
let[@zero_alloc] bar x = [x;x]
88

99
let baz x = [|x;x+1|]
10-
(* CR-soon gyorsh:
11-
The check fails on [baz] with -check-zero-alloc default/all/opt.
12-
This shows that the current interpretation of [@@@zero_alloc all] on functions
13-
that do not have an explicit "zero_alloc" attribute
14-
is to require "zero all" with both "opt" and "default" flags.
15-
This behavior behavior exposes an implementation detail that does not matter in
16-
practice because "opt" flag is only intended for testing, and the behavior
17-
with "default" and "all" flags is indistinguishable.
18-
Fix it in the next commit.
19-
*)

tests/backend/checkmach/test_all_opt3.output

-6
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,3 @@ Error: Annotation check for zero_alloc failed on function Test_all_opt3.foo (cam
33

44
File "test_all_opt3.ml", line 4, characters 29-34:
55
Error: allocation of 24 bytes
6-
7-
File "test_all_opt3.ml", line 9, characters 8-21:
8-
Error: Annotation check for zero_alloc failed on function Test_all_opt3.baz (camlTest_all_opt3.baz_HIDE_STAMP)
9-
10-
File "test_all_opt3.ml", line 9, characters 12-21:
11-
Error: allocation of 24 bytes

0 commit comments

Comments
 (0)