From: Joel Brobecker <brobecker@adacore.com>
Date: Sun, 17 Feb 2019 15:04:57 +0000 (-0500)
Subject: type_align: handle range types the same as ints and enums
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=75ba10dc558a450c7dbcdc24f816f6709f2b29f7;p=binutils-gdb.git

type_align: handle range types the same as ints and enums

This commit enhances type_align to handle TYPE_CODE_RANGE types
the same as integers and enums, rather than returning zero,
which means for this function that it could not determine its
alignment.

gdb/ChangeLog:

	* gdbtypes.c (type_align): Handle TYPE_CODE_RANGE the same as
        integers and enumeration types.

Tested on x86_64-linux. Also tested on a variety of platforms
(with CPUs being ARM, AArch64, Leon3 (SPARC-like), PowerPC,
PowerPC64, RV64, Visium, x86, x86_64).
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index a5945bdc68c..326113a12b8 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2019-02-17  Joel Brobecker  <brobecker@adacore.com>
+
+	* gdbtypes.c (type_align): Handle TYPE_CODE_RANGE the same as
+	integers and enumeration types.
+
 2019-02-17  Joel Brobecker  <brobecker@adacore.com>
 
 	* ada-lang.c (standard_lookup): Use ada_lookup_encoded_symbol
diff --git a/gdb/gdbtypes.c b/gdb/gdbtypes.c
index d1ca304a924..675878337b3 100644
--- a/gdb/gdbtypes.c
+++ b/gdb/gdbtypes.c
@@ -3003,6 +3003,7 @@ type_align (struct type *type)
     case TYPE_CODE_FUNC:
     case TYPE_CODE_FLAGS:
     case TYPE_CODE_INT:
+    case TYPE_CODE_RANGE:
     case TYPE_CODE_FLT:
     case TYPE_CODE_ENUM:
     case TYPE_CODE_REF:
@@ -3047,7 +3048,6 @@ type_align (struct type *type)
       break;
 
     case TYPE_CODE_SET:
-    case TYPE_CODE_RANGE:
     case TYPE_CODE_STRING:
       /* Not sure what to do here, and these can't appear in C or C++
 	 anyway.  */