Skip to content

Commit 50f73cb

Browse files
authored
Do not allow naked pointers (remove configure option) (#3448)
1 parent b7c8ad3 commit 50f73cb

File tree

5 files changed

+7
-32
lines changed

5 files changed

+7
-32
lines changed

configure.ac

+4-29
Original file line numberDiff line numberDiff line change
@@ -357,8 +357,6 @@ AC_SUBST([mkexe_ldflags_exp])
357357
AC_SUBST([PACKLD])
358358
AC_SUBST([build_libraries_manpages])
359359
AC_SUBST([compute_deps])
360-
AC_SUBST([naked_pointers])
361-
AC_SUBST([naked_pointers_checker])
362360
AC_SUBST([intel_jcc_bug_cflags])
363361
AC_SUBST([stack_allocation])
364362
AC_SUBST([poll_insertion])
@@ -585,9 +583,10 @@ AC_ARG_ENABLE([cpp-mangling],
585583
[AS_HELP_STRING([--enable-cpp-mangling],
586584
[use cpp mangling for exported symbols])])
587585

588-
AC_ARG_ENABLE([naked-pointers],
589-
[AS_HELP_STRING([--disable-naked-pointers],
590-
[do not allow naked pointers])])
586+
AC_ARG_ENABLE([naked-pointers], [],
587+
[AS_IF([test "x$enableval" != 'xno'],
588+
[AC_MSG_ERROR([Naked pointers were prohibited in OCaml 5.0.])])],
589+
[])
591590

592591
AC_ARG_ENABLE([naked-pointers-checker], [],
593592
[AS_IF([test "x$enableval" != 'xno'],
@@ -2640,30 +2639,6 @@ AS_IF([test x"$enable_cpp_mangling" = "xyes"],
26402639
AC_DEFINE([WITH_CPP_MANGLING])],
26412640
[cpp_mangling=false])
26422641

2643-
## No naked pointers
2644-
2645-
AS_IF([test x"$enable_naked_pointers" = "xno" ],
2646-
[naked_pointers=false
2647-
AC_DEFINE([NO_NAKED_POINTERS])],
2648-
[naked_pointers=true])
2649-
2650-
AS_IF([test x"$enable_naked_pointers_checker" = "xyes" ],
2651-
[AS_IF([test x"$enable_naked_pointers" = "xno" ],
2652-
[AC_MSG_ERROR(m4_normalize([
2653-
--enable-naked-pointers-checker and --disable-naked-pointers
2654-
are incompatible]))])
2655-
AS_CASE(["$arch","$system"],
2656-
[amd64,linux|amd64,macosx \
2657-
|amd64,openbsd|amd64,win64 \
2658-
|amd64,freebsd|amd64,solaris \
2659-
|arm64,linux|arm64,macosx],
2660-
[naked_pointers_checker=true
2661-
AC_DEFINE([NAKED_POINTERS_CHECKER])],
2662-
[*],
2663-
[AC_MSG_ERROR([naked pointers checker not supported on this platform])]
2664-
)],
2665-
[naked_pointers_checker=false])
2666-
26672642
## Check for mmap support for huge pages and contiguous heap
26682643
OCAML_MMAP_SUPPORTS_HUGE_PAGES
26692644

ocamltest/ocamltest_config.ml.in

+1-1
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,6 @@ let stack_allocation = @stack_allocation@
104104

105105
let poll_insertion = @poll_insertion@
106106

107-
let naked_pointers = @naked_pointers@
107+
let naked_pointers = false
108108

109109
let tsan = @tsan@

runtime4/caml/m.h.in

+1-1
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@
8484

8585
#undef WITH_FRAME_POINTERS
8686

87-
#undef NO_NAKED_POINTERS
87+
#define NO_NAKED_POINTERS 1
8888

8989
#undef NAKED_POINTERS_CHECKER
9090

utils/config.common.ml.in

+1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ and cfg_magic_number = {magic|@CFG_MAGIC_NUMBER@|magic}
4646

4747
let safe_string = true
4848
let default_safe_string = true
49+
let naked_pointers = false
4950
let flambda_backend = true
5051

5152
let interface_suffix = ref ".mli"

utils/config.generated.ml.in

-1
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,6 @@ let flexdll_dirs = [@flexdll_dir@]
9797

9898
let ar_supports_response_files = @ar_supports_response_files@
9999

100-
let naked_pointers = "@naked_pointers@" = "true"
101100
let runtime5 = "@enable_runtime5@" = "yes"
102101

103102
let reserved_header_bits =

0 commit comments

Comments
 (0)