On 11/11/20 4:25 PM, Eric Botcazou wrote: > Hi, > > the Ada language supports fixed-point types as first-class citizens so they > need to be described as-is in the debug info. Pierre-Marie devised the > langhook get_fixed_point_type_info for this purpose a few years ago, but it > comes with a limitation for the representation of the scale factor that we > would need to lift in order to be able to represent more fixed-point types. > > Since it's only used by the Ada compiler, this is probably non-controversial > but someone still needs to approve the change. Thanks in advance. > > > 2020-11-11 Eric Botcazou <ebotca...@adacore.com> > > * dwarf2out.h (struct fixed_point_type_info) <scale_factor>: Turn > numerator and denominator into a tree. > * dwarf2out.c (base_type_die): In the case of a fixed-point type > with arbitrary scale factor, call add_scalar_info on numerator and > denominator to emit the appropriate attributes. > > > 2020-11-11 Eric Botcazou <ebotca...@adacore.com> > > * exp_dbug.adb (Is_Handled_Scale_Factor): Delete. > (Get_Encoded_Name): Do not call it. > * gcc-interface/decl.c (gnat_to_gnu_entity) <Fixed_Point_Type>: > Tidy up and always use a meaningful description for arbitrary > scale factors. > * gcc-interface/misc.c (gnat_get_fixed_point_type_info): Remove > obsolete block and adjust the description of the scale factor.
OK. jeff
