diff options
| author | Xavier Leroy <xavier.leroy@college-de-france.fr> | 2020-12-30 11:00:22 +0100 |
|---|---|---|
| committer | Xavier Leroy <xavierleroy@users.noreply.github.com> | 2021-02-23 11:33:14 +0100 |
| commit | ed89275cb820bb7ab283c51e461d852d1c8bec63 (patch) | |
| tree | e076d92145806c6de92194dfa816e9944c2d27f2 /x86 | |
| parent | 30feb31c6d6e9235acad42ec5d09d14f3919cc36 (diff) | |
| download | compcert-kvx-ed89275cb820bb7ab283c51e461d852d1c8bec63.tar.gz compcert-kvx-ed89275cb820bb7ab283c51e461d852d1c8bec63.zip | |
Section handling: finer control of variable initialization
Distinguish between:
- uninitialized variables, which can go in COMM if supported
- variables initialized with fixed, numeric quantities,
which can go in a readonly section if "const"
- variables initialized with symbol addresses which may need relocation,
which cannot go in a readonly section even if "const",
but can go in a special "const_data" section.
Also: on macOS, use ".const" instead of ".literal8" for literals,
as not all literals have size 8.
Diffstat (limited to 'x86')
| -rw-r--r-- | x86/TargetPrinter.ml | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/x86/TargetPrinter.ml b/x86/TargetPrinter.ml index 102edce4..1b27ee73 100644 --- a/x86/TargetPrinter.ml +++ b/x86/TargetPrinter.ml @@ -194,9 +194,9 @@ module MacOS_System : SYSTEM = | Section_data i | Section_small_data i -> variable_section ~sec:".data" i | Section_const i | Section_small_const i -> - variable_section ~sec:".const" i + variable_section ~sec:".const" ~reloc:".const_data" i | Section_string -> ".const" - | Section_literal -> ".literal8" + | Section_literal -> ".const" | Section_jumptable -> ".text" | Section_user(s, wr, ex) -> sprintf ".section \"%s\", %s, %s" @@ -914,8 +914,7 @@ module Target(System: SYSTEM):TARGET = let print_epilogue oc = if !need_masks then begin - section oc (Section_const true); - (* not Section_literal because not 8-bytes *) + section oc Section_literal; print_align oc 16; fprintf oc "%a: .quad 0x8000000000000000, 0\n" raw_symbol "__negd_mask"; |