DWARF: add a language hook for fixed-point types
authorPierre-Marie de Rodat <derodat@adacore.com>
Thu, 17 Dec 2015 14:09:55 +0000 (14:09 +0000)
committerPierre-Marie de Rodat <pmderodat@gcc.gnu.org>
Thu, 17 Dec 2015 14:09:55 +0000 (14:09 +0000)
Support for fixed-point types in GCC is not powerful enough for Ada
fixed-point types: GNAT uses regular scalar types to implement them.
This new language hook makes it possible to output the desired debugging
information anyway.

gcc/ada/ChangeLog:

* gcc-interface/ada-tree.def (POWER_EXPR): New binary operation.
* gcc-interface/ada-tree.h (TYPE_FIXED_POINT_P): New macro.
(TYPE_IS_FIXED_POINT_P): New macro.
(TYPE_SCALE_FACTOR): New macro.
(SET_TYPE_SCALE_FACTOR): New macro.
* gcc-interface/decl.c: Include urealp.h
(gnat_to_gnu_entity): Attach trees to encode scale factors to
fixed-point types.
* gcc-interface/misc.c (gnat_print_type): Print scale factors
for fixed-point types.
(gnat_get_fixed_point_type_info): New.
(gnat_init_ts): Initialize data for the POWER_EXPR binary
operation.
(LANG_HOOKS_GET_FIXED_POINT_INFO): Redefine macro to implement
the get_fixed_point_type_info language hook.

gcc/ChangeLog:

* langhooks.h (struct lang_hooks_for_types): Add a
get_fixed_point_type_info field.
* langhooks-def.h (LANG_HOOKS_GET_FIXED_POINT_TYPE_INFO): New
macro.
(LANG_HOOKS_FOR_TYPES_INITIALIZER): Initialize the
get_fixed_point_type_info field.
* dwarf2out.h (enum fixed_point_scale_factor): New.
(struct fixed_point_type_info): New.
* dwarf2out.c (base_type_die): In DWARFv3 or non-strict DWARF
mode, get fixed-point type information using the debugging hook
and describe it in DWARF, if any.

From-SVN: r231764

gcc/ChangeLog
gcc/ada/ChangeLog
gcc/ada/gcc-interface/ada-tree.def
gcc/ada/gcc-interface/ada-tree.h
gcc/ada/gcc-interface/decl.c
gcc/ada/gcc-interface/misc.c
gcc/dwarf2out.c
gcc/dwarf2out.h
gcc/langhooks-def.h
gcc/langhooks.h

index 94b772607d0b1037cd0a623348b16caf3fcfe4a5..964b0a375af5815b4525ba034973127b982d563e 100644 (file)
@@ -1,3 +1,17 @@
+2015-12-17  Pierre-Marie de Rodat  <derodat@adacore.com>
+
+       * langhooks.h (struct lang_hooks_for_types): Add a
+       get_fixed_point_type_info field.
+       * langhooks-def.h (LANG_HOOKS_GET_FIXED_POINT_TYPE_INFO): New
+       macro.
+       (LANG_HOOKS_FOR_TYPES_INITIALIZER): Initialize the
+       get_fixed_point_type_info field.
+       * dwarf2out.h (enum fixed_point_scale_factor): New.
+       (struct fixed_point_type_info): New.
+       * dwarf2out.c (base_type_die): In DWARFv3 or non-strict DWARF
+       mode, get fixed-point type information using the debugging hook
+       and describe it in DWARF, if any.
+
 2015-12-17  Pierre-Marie de Rodat  <derodat@adacore.com>
 
        * langhooks.h (struct lang_hooks_for_types): Add a
index 8c6a56d7f1a4f7d9696c853be8e8b48d66d950bb..31a059f8e25594b90cd64a12b0edac23cfb48a06 100644 (file)
@@ -1,3 +1,21 @@
+2015-12-17  Pierre-Marie de Rodat  <derodat@adacore.com>
+
+       * gcc-interface/ada-tree.def (POWER_EXPR): New binary operation.
+       * gcc-interface/ada-tree.h (TYPE_FIXED_POINT_P): New macro.
+       (TYPE_IS_FIXED_POINT_P): New macro.
+       (TYPE_SCALE_FACTOR): New macro.
+       (SET_TYPE_SCALE_FACTOR): New macro.
+       * gcc-interface/decl.c: Include urealp.h
+       (gnat_to_gnu_entity): Attach trees to encode scale factors to
+       fixed-point types.
+       * gcc-interface/misc.c (gnat_print_type): Print scale factors
+       for fixed-point types.
+       (gnat_get_fixed_point_type_info): New.
+       (gnat_init_ts): Initialize data for the POWER_EXPR binary
+       operation.
+       (LANG_HOOKS_GET_FIXED_POINT_INFO): Redefine macro to implement
+       the get_fixed_point_type_info language hook.
+
 2015-12-17  Pierre-Marie de Rodat  <derodat@adacore.com>
 
        * gcc-interface/ada-tree.h (struct lang_type): Rename the t
index 93967b58cb33622820a9d762f0f798f46d9a4d2f..8eb4688e749cbf7381b19ae29acc691830249383 100644 (file)
@@ -47,6 +47,11 @@ DEFTREECODE (PLUS_NOMOD_EXPR, "plus_nomod_expr", tcc_binary, 2)
    This is used for loops and never shows up in the tree.  */
 DEFTREECODE (MINUS_NOMOD_EXPR, "minus_nomod_expr", tcc_binary, 2)
 
+/* An expression that computes an exponentiation.  Operand 0 is the base and
+   Operand 1 is the exponent.  This node is never passed to GCC: it is only
+   used internally to describe fixed point types scale factors.  */
+DEFTREECODE (POWER_EXPR, "power_expr", tcc_binary, 2)
+
 /* Same as ADDR_EXPR, except that if the operand represents a bit field,
    return the address of the byte containing the bit.  This is used
    for the Address attribute and never shows up in the tree.  */
index 807da9d97cff3c387c50a19b64f9088fd9e767f5..709fdc27061b7654c2591cd1f2ea69f97fb41600 100644 (file)
@@ -126,6 +126,13 @@ do {                                                        \
 #define TYPE_CONTAINS_TEMPLATE_P(NODE) \
   TYPE_LANG_FLAG_3 (RECORD_OR_UNION_CHECK (NODE))
 
+/* For INTEGER_TYPE, nonzero if it implements a fixed-point type.  */
+#define TYPE_FIXED_POINT_P(NODE) \
+  TYPE_LANG_FLAG_3 (INTEGER_TYPE_CHECK (NODE))
+
+#define TYPE_IS_FIXED_POINT_P(NODE) \
+  (TREE_CODE (NODE) == INTEGER_TYPE && TYPE_FIXED_POINT_P (NODE))
+
 /* True if NODE is a thin pointer.  */
 #define TYPE_IS_THIN_POINTER_P(NODE)                   \
   (POINTER_TYPE_P (NODE)                               \
@@ -358,6 +365,16 @@ do {                                                  \
 #define SET_TYPE_DEBUG_TYPE(NODE, X) \
   SET_TYPE_LANG_SPECIFIC2(NODE, X)
 
+/* For an INTEGER_TYPE with TYPE_IS_FIXED_POINT_P, this is the value of the
+   scale factor.  Modular types, index types (sizetype subtypes) and
+   fixed-point types are totally distinct types, so there is no problem with
+   sharing type lang specific's first slot.  */
+#define TYPE_SCALE_FACTOR(NODE) \
+  GET_TYPE_LANG_SPECIFIC (INTEGER_TYPE_CHECK (NODE))
+#define SET_TYPE_SCALE_FACTOR(NODE, X) \
+  SET_TYPE_LANG_SPECIFIC (INTEGER_TYPE_CHECK (NODE), X)
+
+
 /* Flags added to decl nodes.  */
 
 /* Nonzero in a FUNCTION_DECL that represents a stubbed function
index 385b720506a19fce621a201cab93c3d0c11339d1..760c7f439f14b956f9ace85f4c62c0106726cd73 100644 (file)
@@ -44,6 +44,7 @@
 #include "repinfo.h"
 #include "snames.h"
 #include "uintp.h"
+#include "urealp.h"
 #include "fe.h"
 #include "sinfo.h"
 #include "einfo.h"
@@ -1619,13 +1620,80 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
       break;
 
     case E_Signed_Integer_Type:
-    case E_Ordinary_Fixed_Point_Type:
-    case E_Decimal_Fixed_Point_Type:
       /* For integer types, just make a signed type the appropriate number
         of bits.  */
       gnu_type = make_signed_type (esize);
       goto discrete_type;
 
+    case E_Ordinary_Fixed_Point_Type:
+    case E_Decimal_Fixed_Point_Type:
+      {
+       /* Small_Value is the scale factor.  */
+       const Ureal gnat_small_value = Small_Value (gnat_entity);
+       tree scale_factor = NULL_TREE;
+
+       gnu_type = make_signed_type (esize);
+
+       /* Try to decode the scale factor and to save it for the fixed-point
+          types debug hook.  */
+
+       /* There are various ways to describe the scale factor, however there
+          are cases where back-end internals cannot hold it.  In such cases,
+          we output invalid scale factor for such cases (i.e. the 0/0
+          rational constant) but we expect GNAT to output GNAT encodings,
+          then.  Thus, keep this in sync with
+          Exp_Dbug.Is_Handled_Scale_Factor.  */
+
+       /* When encoded as 1/2**N or 1/10**N, describe the scale factor as a
+          binary or decimal scale: it is easier to read for humans.  */
+       if (UI_Eq (Numerator (gnat_small_value), Uint_1)
+           && (Rbase (gnat_small_value) == 2
+               || Rbase (gnat_small_value) == 10))
+         {
+           /* Given RM restrictions on 'Small values, we assume here that
+              the denominator fits in an int.  */
+           const tree base = build_int_cst (integer_type_node,
+                                            Rbase (gnat_small_value));
+           const tree exponent
+             = build_int_cst (integer_type_node,
+                              UI_To_Int (Denominator (gnat_small_value)));
+           scale_factor
+             = build2 (RDIV_EXPR, integer_type_node,
+                       integer_one_node,
+                       build2 (POWER_EXPR, integer_type_node,
+                               base, exponent));
+         }
+
+       /* Default to arbitrary scale factors descriptions.  */
+       else
+         {
+           const Uint num = Norm_Num (gnat_small_value);
+           const Uint den = Norm_Den (gnat_small_value);
+
+           if (UI_Is_In_Int_Range (num) && UI_Is_In_Int_Range (den))
+             {
+               const tree gnu_num
+                 = build_int_cst (integer_type_node,
+                                  UI_To_Int (Norm_Num (gnat_small_value)));
+               const tree gnu_den
+                 = build_int_cst (integer_type_node,
+                                  UI_To_Int (Norm_Den (gnat_small_value)));
+               scale_factor = build2 (RDIV_EXPR, integer_type_node,
+                                      gnu_num, gnu_den);
+             }
+           else
+             /* If compiler internals cannot represent arbitrary scale
+                factors, output an invalid scale factor so that debugger
+                don't try to handle them but so that we still have a type
+                in the output.  Note that GNAT  */
+             scale_factor = integer_zero_node;
+         }
+
+       TYPE_FIXED_POINT_P (gnu_type) = 1;
+       SET_TYPE_SCALE_FACTOR (gnu_type, scale_factor);
+      }
+      goto discrete_type;
+
     case E_Modular_Integer_Type:
       {
        /* For modular types, make the unsigned type of the proper number
index e9df63c1fc75d3c46e19be6e8d283127ea796bf1..48e98fd583c0e829298f8c5215b2a4ddc711cd68 100644 (file)
@@ -480,6 +480,9 @@ gnat_print_type (FILE *file, tree node, int indent)
     case INTEGER_TYPE:
       if (TYPE_MODULAR_P (node))
        print_node_brief (file, "modulus", TYPE_MODULUS (node), indent + 4);
+      else if (TYPE_FIXED_POINT_P (node))
+       print_node (file, "scale factor", TYPE_SCALE_FACTOR (node),
+                   indent + 4);
       else if (TYPE_HAS_ACTUAL_BOUNDS_P (node))
        print_node (file, "actual bounds", TYPE_ACTUAL_BOUNDS (node),
                    indent + 4);
@@ -578,6 +581,81 @@ gnat_get_debug_type (const_tree type)
   return TYPE_DEBUG_TYPE (type);
 }
 
+/* Provide information in INFO for debugging output about the TYPE fixed-point
+   type.  Return whether TYPE is handled.  */
+
+static bool
+gnat_get_fixed_point_type_info (const_tree type,
+                               struct fixed_point_type_info *info)
+{
+  tree scale_factor;
+
+  /* GDB cannot handle fixed-point types yet, so rely on GNAT encodings
+     instead for it.  */
+  if (gnat_encodings != DWARF_GNAT_ENCODINGS_MINIMAL
+      || !TYPE_IS_FIXED_POINT_P (type))
+    return false;
+
+  scale_factor = TYPE_SCALE_FACTOR (type);
+
+  /* We expect here only a finite set of pattern.  See fixed-point types
+     handling in gnat_to_gnu_entity.  */
+
+  /* Put invalid values when compiler internals cannot represent the scale
+     factor.  */
+  if (scale_factor == integer_zero_node)
+    {
+      info->scale_factor_kind = fixed_point_scale_factor_arbitrary;
+      info->scale_factor.arbitrary.numerator = 0;
+      info->scale_factor.arbitrary.denominator = 0;
+      return true;
+    }
+
+  if (TREE_CODE (scale_factor) == RDIV_EXPR)
+    {
+      const tree num = TREE_OPERAND (scale_factor, 0);
+      const tree den = TREE_OPERAND (scale_factor, 1);
+
+      /* See if we have a binary or decimal scale.  */
+      if (TREE_CODE (den) == POWER_EXPR)
+       {
+         const tree base = TREE_OPERAND (den, 0);
+         const tree exponent = TREE_OPERAND (den, 1);
+
+         /* We expect the scale factor to be 1 / 2 ** N or 1 / 10 ** N.  */
+         gcc_assert (num == integer_one_node
+                     && TREE_CODE (base) == INTEGER_CST
+                     && TREE_CODE (exponent) == INTEGER_CST);
+         switch (tree_to_shwi (base))
+           {
+           case 2:
+             info->scale_factor_kind = fixed_point_scale_factor_binary;
+             info->scale_factor.binary = -tree_to_shwi (exponent);
+             return true;
+
+           case 10:
+             info->scale_factor_kind = fixed_point_scale_factor_decimal;
+             info->scale_factor.decimal = -tree_to_shwi (exponent);
+             return true;
+
+           default:
+             gcc_unreachable ();
+           }
+       }
+
+      /* If we reach this point, we are handling an arbitrary scale factor.  We
+        expect N / D with constant operands.  */
+      gcc_assert (TREE_CODE (num) == INTEGER_CST
+                 && TREE_CODE (den) == INTEGER_CST);
+      info->scale_factor_kind = fixed_point_scale_factor_arbitrary;
+      info->scale_factor.arbitrary.numerator = tree_to_uhwi (num);
+      info->scale_factor.arbitrary.denominator = tree_to_shwi (den);
+      return true;
+    }
+
+  gcc_unreachable ();
+}
+
 /* Return true if types T1 and T2 are identical for type hashing purposes.
    Called only after doing all language independent checks.  At present,
    this function is only called when both types are FUNCTION_TYPE.  */
@@ -981,6 +1059,7 @@ gnat_init_ts (void)
   MARK_TS_TYPED (NULL_EXPR);
   MARK_TS_TYPED (PLUS_NOMOD_EXPR);
   MARK_TS_TYPED (MINUS_NOMOD_EXPR);
+  MARK_TS_TYPED (POWER_EXPR);
   MARK_TS_TYPED (ATTR_ADDR_EXPR);
   MARK_TS_TYPED (STMT_STMT);
   MARK_TS_TYPED (LOOP_STMT);
@@ -1052,6 +1131,9 @@ get_lang_specific (tree node)
 #define LANG_HOOKS_DESCRIPTIVE_TYPE    gnat_descriptive_type
 #undef  LANG_HOOKS_GET_DEBUG_TYPE
 #define LANG_HOOKS_GET_DEBUG_TYPE      gnat_get_debug_type
+#undef  LANG_HOOKS_GET_FIXED_POINT_TYPE_INFO
+#define LANG_HOOKS_GET_FIXED_POINT_TYPE_INFO \
+                                       gnat_get_fixed_point_type_info
 #undef  LANG_HOOKS_ATTRIBUTE_TABLE
 #define LANG_HOOKS_ATTRIBUTE_TABLE     gnat_internal_attribute_table
 #undef  LANG_HOOKS_BUILTIN_FUNCTION
index 800f283aa319b5f9741ffddf8893a9564e881732..538b76dd8788afba1337f68a04e52715e296dcc5 100644 (file)
@@ -10807,6 +10807,8 @@ base_type_die (tree type)
 {
   dw_die_ref base_type_result;
   enum dwarf_type encoding;
+  bool fpt_used = false;
+  struct fixed_point_type_info fpt_info;
 
   if (TREE_CODE (type) == ERROR_MARK || TREE_CODE (type) == VOID_TYPE)
     return 0;
@@ -10833,6 +10835,19 @@ base_type_die (tree type)
              break;
            }
        }
+      if ((dwarf_version >= 3 || !dwarf_strict)
+         && lang_hooks.types.get_fixed_point_type_info)
+       {
+         memset (&fpt_info, 0, sizeof (fpt_info));
+         if (lang_hooks.types.get_fixed_point_type_info (type, &fpt_info))
+           {
+             fpt_used = true;
+             encoding = ((TYPE_UNSIGNED (type))
+                         ? DW_ATE_unsigned_fixed
+                         : DW_ATE_signed_fixed);
+             break;
+           }
+       }
       if (TYPE_STRING_FLAG (type))
        {
          if (TYPE_UNSIGNED (type))
@@ -10891,6 +10906,43 @@ base_type_die (tree type)
   add_AT_unsigned (base_type_result, DW_AT_byte_size,
                   int_size_in_bytes (type));
   add_AT_unsigned (base_type_result, DW_AT_encoding, encoding);
+
+  if (fpt_used)
+    {
+      switch (fpt_info.scale_factor_kind)
+       {
+       case fixed_point_scale_factor_binary:
+         add_AT_int (base_type_result, DW_AT_binary_scale,
+                     fpt_info.scale_factor.binary);
+         break;
+
+       case fixed_point_scale_factor_decimal:
+         add_AT_int (base_type_result, DW_AT_decimal_scale,
+                     fpt_info.scale_factor.decimal);
+         break;
+
+       case fixed_point_scale_factor_arbitrary:
+         /* Arbitrary scale factors cannot be described in standard DWARF,
+            yet.  */
+         if (!dwarf_strict)
+           {
+             /* Describe the scale factor as a rational constant.  */
+             const dw_die_ref scale_factor
+               = new_die (DW_TAG_constant, comp_unit_die (), type);
+
+             add_AT_unsigned (scale_factor, DW_AT_GNU_numerator,
+                              fpt_info.scale_factor.arbitrary.numerator);
+             add_AT_int (scale_factor, DW_AT_GNU_denominator,
+                         fpt_info.scale_factor.arbitrary.denominator);
+
+             add_AT_die_ref (base_type_result, DW_AT_small, scale_factor);
+           }
+         break;
+
+       default:
+         gcc_unreachable ();
+       }
+    }
   add_pubtype (type, base_type_result);
 
   return base_type_result;
index 7f4a24d977d86eced807fa50b41476c783d43259..91730766bebf2843396f07cbb5dacdc911a7de71 100644 (file)
@@ -342,6 +342,35 @@ struct array_descr_info
     } dimen[10];
 };
 
+enum fixed_point_scale_factor
+{
+  fixed_point_scale_factor_binary,
+  fixed_point_scale_factor_decimal,
+  fixed_point_scale_factor_arbitrary
+};
+
+struct fixed_point_type_info
+{
+  /* A scale factor is the value one has to multiply with physical data in
+     order to get the fixed point logical data.  The DWARF standard enables one
+     to encode it in three ways.  */
+  enum fixed_point_scale_factor scale_factor_kind;
+  union
+    {
+      /* For binary scale factor, the scale factor is: 2 ** binary.  */
+      int binary;
+      /* For decimal scale factor, the scale factor is: 10 ** binary.  */
+      int decimal;
+      /* For arbitrary scale factor, the scale factor is:
+        numerator / denominator.  */
+      struct
+       {
+         unsigned HOST_WIDE_INT numerator;
+         HOST_WIDE_INT denominator;
+       } arbitrary;
+    } scale_factor;
+};
+
 void dwarf2out_c_finalize (void);
 
 #endif /* GCC_DWARF2OUT_H */
index 1eafed6155a092800b875466960f863c1f783feb..2d02bf631d018d46ef881b78ddb92559201d1d11 100644 (file)
@@ -177,6 +177,7 @@ extern tree lhd_make_node (enum tree_code);
 #define LANG_HOOKS_RECONSTRUCT_COMPLEX_TYPE reconstruct_complex_type
 #define LANG_HOOKS_ENUM_UNDERLYING_BASE_TYPE lhd_enum_underlying_base_type
 #define LANG_HOOKS_GET_DEBUG_TYPE      NULL
+#define LANG_HOOKS_GET_FIXED_POINT_TYPE_INFO NULL
 
 #define LANG_HOOKS_FOR_TYPES_INITIALIZER { \
   LANG_HOOKS_MAKE_TYPE, \
@@ -197,7 +198,8 @@ extern tree lhd_make_node (enum tree_code);
   LANG_HOOKS_DESCRIPTIVE_TYPE, \
   LANG_HOOKS_RECONSTRUCT_COMPLEX_TYPE, \
   LANG_HOOKS_ENUM_UNDERLYING_BASE_TYPE, \
-  LANG_HOOKS_GET_DEBUG_TYPE \
+  LANG_HOOKS_GET_DEBUG_TYPE, \
+  LANG_HOOKS_GET_FIXED_POINT_TYPE_INFO \
 }
 
 /* Declaration hooks.  */
index e80993b8963907d56c62b34c15db00629f15a81a..f84bdf64bf4d95116bd3fd796b1dd4eeab10b6a7 100644 (file)
@@ -149,6 +149,11 @@ struct lang_hooks_for_types
      middle-end uses specialized types, for instance constrained discriminated
      types in Ada.  */
   tree (*get_debug_type) (const_tree);
+
+  /* Return TRUE if TYPE implements a fixed point type and fills in information
+     for the debugger about scale factor, etc.  */
+  bool (*get_fixed_point_type_info) (const_tree,
+                                    struct fixed_point_type_info *);
 };
 
 /* Language hooks related to decls and the symbol table.  */