Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improve --heap-size-hint arg handling #48050

Merged
merged 15 commits into from
Feb 23, 2024
Merged

improve --heap-size-hint arg handling #48050

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
c4c5229
improve heap-size-hint arg handling
mbauman Dec 30, 2022
da564a7
more testsets
mbauman Dec 30, 2022
e7fb5dd
less undefined behavior, more debugging
mbauman Jan 4, 2023
84cbd0e
better debugging
mbauman Jan 5, 2023
fa2d281
long double consistently
mbauman Jan 5, 2023
3b889d4
Merge branch 'master' into mb/heap_size_hint_cleanup
gbaraldi Jan 16, 2023
35dc093
Merge remote-tracking branch 'origin/master' into HEAD
mbauman Feb 8, 2024
5f32478
remove debugging code
mbauman Feb 8, 2024
7495e55
match verbiage with #50480
mbauman Feb 8, 2024
2974896
standardize documentation
mbauman Feb 9, 2024
1291220
use utf8proc_tolower
mbauman Feb 12, 2024
a7e1bb3
Merge branch 'master' into mb/heap_size_hint_cleanup
vtjnash Feb 13, 2024
07fbf57
use a static ascii_tolower
mbauman Feb 19, 2024
3de28b5
Merge remote-tracking branch 'origin/master' into mb/heap_size_hint_c…
mbauman Feb 20, 2024
b902aab
Merge branch 'master' into mb/heap_size_hint_cleanup
mbauman Feb 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion src/jloptions.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -809,7 +809,11 @@ JL_DLLEXPORT void jl_parse_opts(int *argcp, char ***argvp)
jl_errorf("julia: invalid argument to --heap-size-hint (%s)", optarg);
break;
}
jl_options.heap_size_hint = (uint64_t)(value * multiplier);
double sz = value * multiplier;
vtjnash marked this conversation as resolved.
Show resolved Hide resolved
mbauman marked this conversation as resolved.
Show resolved Hide resolved
if (isnan(sz) || sz < 0) {
jl_errorf("julia: invalid argument to --heap-size-hint (%s)", optarg);
}
jl_options.heap_size_hint = sz < UINT64_MAX ? (uint64_t)sz : UINT64_MAX;
}
if (jl_options.heap_size_hint == 0)
jl_errorf("julia: invalid argument to --heap-size-hint without memory size specified");
Expand Down
6 changes: 5 additions & 1 deletion test/cmdlineargs.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -832,8 +832,12 @@ end
@testset "--heap-size-hint" begin
exename = `$(Base.julia_cmd())`
@test errors_not_signals(`$exename --heap-size-hint -e "exit(0)"`)
@testset "--heap-size-hint=$str" for str in ["asdf", "", "0","1.2vb","b","GB","1.2gb2","42gigabytes","5gig","2GiB","NaNt"]
@testset "--heap-size-hint=$str" for str in ["asdf","","0","1.2vb","b","GB","1.2gb2","42gigabytes","5gig","2GiB","NaNt"]
@test errors_not_signals(`$exename --heap-size-hint=$str -e "exit(0)"`)
if errors_not_signals(`$exename --heap-size-hint=$str -e "exit(0)"`)
@show str
run(`$exename --heap-size-hint=$str -e "exit(0)"`)
end
end
k = 1024
m = 1024k
Expand Down