Address review comments for the previous series

I failed at git and missed adding/lost changes on the wrong branch, the
result being that I didn't incorporate fixes resulting from Yao's review
comments.  This patch fixes that.

There are two places where we should use the unique pointer typedef, and
ChangeLog entries missing.

gdb/ChangeLog:

	* target-descriptions.c (struct tdesc_feature) <registers>: Use
	tdesc_reg_up typedef.
	(struct target_desc) <features>: Use tdesc_feature_up typedef.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1d3a3e8104264e96d29d4b91a3c2ced129f7222d..657f87b3b7d62bcc2c9da60d11b0fafe8200771d 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2017-12-05  Simon Marchi  <simon.marchi@ericsson.com>
+
+       * target-descriptions.c (struct tdesc_feature) <registers>: Use
+       tdesc_reg_up typedef.
+       (struct target_desc) <features>: Use tdesc_feature_up typedef.
+
 2017-12-05  Simon Marchi  <simon.marchi@polymtl.ca>
 
        * target-descriptions.c (struct tdesc_type): Use default
@@ -118,6 +124,8 @@
        (tdesc_register_in_reggroup_p): Adjust.
        (class print_c_tdesc) <visit>: Adjust.
        (class print_c_feature) <visit>: Adjust.
+       * features/arc-arcompact.c: Re-generate.
+       * features/arc-v2.c: Re-generate.
 
 2017-12-05  Simon Marchi  <simon.marchi@ericsson.com>
 

diff --git a/gdb/target-descriptions.c b/gdb/target-descriptions.c
index 12d72fa128e4da9cd3efc4cae1c9f58d1f9a6289..5a6f61945dd165db5136947ae3aa91bde14de1ba 100644 (file)
--- a/gdb/target-descriptions.c
+++ b/gdb/target-descriptions.c
@@ -529,7 +529,7 @@ struct tdesc_feature : tdesc_element
   std::string name;
 
   /* The registers associated with this feature.  */
-  std::vector<std::unique_ptr<tdesc_reg>> registers;
+  std::vector<tdesc_reg_up> registers;
 
   /* The types associated with this feature.  */
   std::vector<tdesc_type_up> types;
@@ -613,7 +613,7 @@ struct target_desc : tdesc_element
   std::vector<property> properties;
 
   /* The features associated with this target.  */
-  std::vector<std::unique_ptr<tdesc_feature>> features;
+  std::vector<tdesc_feature_up> features;
 
   void accept (tdesc_element_visitor &v) const override
   {