From 9fcb0316df79ee741272340d7db0378872a53c5f Mon Sep 17 00:00:00 2001 From: Bernhard Schommer Date: Thu, 28 Jul 2022 18:10:30 +0200 Subject: Rework of struct member offsets for debug info. The struct member byte and bit offsets are now set based upon the new function `struct_layout` from Ctypes.v, thus using the same code to compute as used to generate the actual struct access. The struct offset member information is addded using the types computed after the translation in C2C. Therefore we need to store the new internal names of the members as well as the composites and use them when adding the offset information. Fixes: #445 --- debug/Debug.ml | 22 +++++++++++++--------- debug/Debug.mli | 14 ++++++++------ debug/DebugInformation.ml | 36 +++++++++++++++++++++++++----------- debug/DebugTypes.mli | 2 +- 4 files changed, 47 insertions(+), 27 deletions(-) (limited to 'debug') diff --git a/debug/Debug.ml b/debug/Debug.ml index 812f57cc..a1a68b74 100644 --- a/debug/Debug.ml +++ b/debug/Debug.ml @@ -23,9 +23,10 @@ type implem = { init: string -> unit; atom_global: ident -> atom -> unit; - set_composite_size: ident -> struct_or_union -> int option -> unit; - set_member_offset: ident -> string -> int -> unit; - set_bitfield_offset: ident -> string -> int -> string -> int -> unit; + set_composite_size: ident -> atom -> struct_or_union -> int option -> unit; + set_member_atom: str_id:atom -> string -> fld_id:atom -> unit; + set_member_offset: str_id:atom -> fld_id:atom -> int -> unit; + set_bitfield_offset: str_id:atom -> fld_id:atom -> bit_ofs:int -> byte_ofs:int -> size:int -> unit; insert_global_declaration: Env.t -> globdecl -> unit; add_fun_addr: atom -> section_name -> (int * int) -> unit; generate_debug_info: (atom -> string) -> string -> debug_entries option; @@ -55,9 +56,10 @@ let default_implem = { init = (fun _ -> ()); atom_global = (fun _ _ -> ()); - set_composite_size = (fun _ _ _ -> ()); - set_member_offset = (fun _ _ _ -> ()); - set_bitfield_offset = (fun _ _ _ _ _ -> ()); + set_composite_size = (fun _ _ _ _ -> ()); + set_member_atom = (fun ~str_id _ ~fld_id -> ()); + set_member_offset = (fun ~str_id ~fld_id _ -> ()); + set_bitfield_offset = (fun ~str_id ~fld_id ~bit_ofs ~byte_ofs ~size-> ()); insert_global_declaration = (fun _ _ -> ()); add_fun_addr = (fun _ _ _ -> ()); generate_debug_info = (fun _ _ -> None); @@ -87,9 +89,11 @@ let implem = ref default_implem let init_compile_unit name = !implem.init name let atom_global id atom = !implem.atom_global id atom -let set_composite_size id sou size = !implem.set_composite_size id sou size -let set_member_offset id field off = !implem.set_member_offset id field off -let set_bitfield_offset id field off underlying size = !implem.set_bitfield_offset id field off underlying size +let set_composite_size id atom sou size = !implem.set_composite_size id atom sou size +let set_member_atom ~str_id field ~fld_id = !implem.set_member_atom ~str_id:str_id field ~fld_id:fld_id +let set_member_offset ~str_id ~fld_id off = !implem.set_member_offset ~str_id:str_id ~fld_id:fld_id off +let set_bitfield_offset ~str_id ~fld_id ~bit_ofs ~byte_ofs ~size = + !implem.set_bitfield_offset ~str_id:str_id ~fld_id:fld_id ~bit_ofs:bit_ofs ~byte_ofs:byte_ofs ~size:size let insert_global_declaration env dec = !implem.insert_global_declaration env dec let add_fun_addr atom addr = !implem.add_fun_addr atom addr let generate_debug_info fun_s var_s = !implem.generate_debug_info fun_s var_s diff --git a/debug/Debug.mli b/debug/Debug.mli index 60e2f9bc..001cf0ea 100644 --- a/debug/Debug.mli +++ b/debug/Debug.mli @@ -22,9 +22,10 @@ type implem = { init: string -> unit; atom_global: ident -> atom -> unit; - set_composite_size: ident -> struct_or_union -> int option -> unit; - set_member_offset: ident -> string -> int -> unit; - set_bitfield_offset: ident -> string -> int -> string -> int -> unit; + set_composite_size: ident -> atom -> struct_or_union -> int option -> unit; + set_member_atom: str_id:atom -> string -> fld_id:atom -> unit; + set_member_offset: str_id:atom -> fld_id:atom -> int -> unit; + set_bitfield_offset: str_id:atom -> fld_id:atom -> bit_ofs:int -> byte_ofs:int -> size:int -> unit; insert_global_declaration: Env.t -> globdecl -> unit; add_fun_addr: atom -> section_name -> (int * int) -> unit; generate_debug_info: (atom -> string) -> string -> debug_entries option; @@ -56,9 +57,10 @@ val implem: implem ref val init_compile_unit: string -> unit val atom_global: ident -> atom -> unit -val set_composite_size: ident -> struct_or_union -> int option -> unit -val set_member_offset: ident -> string -> int -> unit -val set_bitfield_offset: ident -> string -> int -> string -> int -> unit +val set_composite_size: ident -> atom -> struct_or_union -> int option -> unit +val set_member_atom: str_id:atom -> string -> fld_id:atom -> unit +val set_member_offset: str_id:atom -> fld_id:atom -> int -> unit +val set_bitfield_offset: str_id:atom -> fld_id:atom -> bit_ofs:int -> byte_ofs:int -> size:int -> unit val insert_global_declaration: Env.t -> globdecl -> unit val add_fun_addr: atom -> section_name -> (int * int) -> unit val all_files_iter: (string -> unit) -> unit diff --git a/debug/DebugInformation.ml b/debug/DebugInformation.ml index f9684355..f6e2e271 100644 --- a/debug/DebugInformation.ml +++ b/debug/DebugInformation.ml @@ -396,13 +396,13 @@ let insert_global_declaration env dec = let fields = List.map (fun f -> { cfd_name = f.fld_name; + cfd_atom = None; cfd_anon = f.fld_anonymous; cfd_typ = insert_type f.fld_typ; cfd_bit_size = f.fld_bitfield; cfd_bit_offset = None; cfd_byte_offset = None; cfd_byte_size = None; - cfd_bitfield = None; }) fi in replace_composite id (fun comp -> let loc = if comp.ct_file_loc = None then Some dec.gloc else comp.ct_file_loc in @@ -423,23 +423,36 @@ let insert_global_declaration env dec = {en with enum_file_loc = Some dec.gloc; enum_enumerators = enumerator;}) | Gpragma _ -> () -let set_member_offset str field offset = - let id = find_type (TStruct (str,[])) in +let atom_is_member id f = + match f.cfd_atom with + | None -> false + | Some f -> f = id + +let set_member_atom ~str_id field ~fld_id = + let sid = Hashtbl.find atom_to_definition str_id in + replace_composite sid (fun comp -> + let name f = f.cfd_name = field in + let members = list_replace name (fun a -> {a with cfd_atom = Some fld_id;}) comp.ct_members in + {comp with ct_members = members;}) + +let set_member_offset ~str_id ~fld_id offset = + let id = Hashtbl.find atom_to_definition str_id in replace_composite id (fun comp -> - let name f = f.cfd_name = field || match f.cfd_bitfield with Some n -> n = field | _ -> false in - let members = list_replace name (fun a -> {a with cfd_byte_offset = Some offset;}) comp.ct_members in + let members = list_replace (atom_is_member fld_id) (fun a -> {a with cfd_byte_offset = Some offset;}) comp.ct_members in {comp with ct_members = members;}) -let set_composite_size comp sou size = +let set_composite_size comp intern_name sou size = let id = find_type (gen_comp_typ sou comp []) in + Hashtbl.add atom_to_definition intern_name id; replace_composite id (fun comp -> {comp with ct_sizeof = size;}) -let set_bitfield_offset str field offset underlying size = - let id = find_type (TStruct (str,[])) in +let set_bitfield_offset ~str_id ~fld_id ~bit_ofs ~byte_ofs ~size = + let id = Hashtbl.find atom_to_definition str_id in replace_composite id (fun comp -> - let name f = f.cfd_name = field in - let members = list_replace name (fun a -> - {a with cfd_bit_offset = Some offset; cfd_bitfield = Some underlying; cfd_byte_size = Some size}) + let members = list_replace (atom_is_member fld_id) (fun a -> + {a with cfd_bit_offset = Some bit_ofs; + cfd_byte_size = Some size; + cfd_byte_offset = Some byte_ofs}) comp.ct_members in {comp with ct_members = members;}) @@ -670,6 +683,7 @@ let default_debug = init = init; atom_global = atom_global; set_composite_size = set_composite_size; + set_member_atom = set_member_atom; set_member_offset = set_member_offset; set_bitfield_offset = set_bitfield_offset; insert_global_declaration = insert_global_declaration; diff --git a/debug/DebugTypes.mli b/debug/DebugTypes.mli index c3df6066..b3002f4c 100644 --- a/debug/DebugTypes.mli +++ b/debug/DebugTypes.mli @@ -19,13 +19,13 @@ open Camlcoq type composite_field = { cfd_name: string; + cfd_atom: atom option; cfd_anon: bool; cfd_typ: int; cfd_bit_size: int option; cfd_bit_offset: int option; cfd_byte_offset: int option; cfd_byte_size: int option; - cfd_bitfield: string option; } type composite_type = -- cgit