[gcc(refs/vendors/ARM/heads/arm-perf-staging)] calls: Remove FIXME for cxx17_empty_base_field_p

Tamar Christina tnfchris@gcc.gnu.org
Fri Jul 17 15:04:50 GMT 2020


https://gcc.gnu.org/g:3bce790414afaaba10e03f7bb41e4aa3279694f5

commit 3bce790414afaaba10e03f7bb41e4aa3279694f5
Author: Richard Sandiford <richard.sandiford@arm.com>
Date:   Wed Apr 29 13:47:21 2020 +0100

    calls: Remove FIXME for cxx17_empty_base_field_p
    
    This predicate is now used by aarch64 targets.
    
    2020-04-29  Richard Sandiford  <richard.sandiford@arm.com>
    
    gcc/
            * calls.h (cxx17_empty_base_field_p): Turn into a function declaration.
            * calls.c (cxx17_empty_base_field_p): New function.  Check
            DECL_ARTIFICIAL and RECORD_OR_UNION_TYPE_P in addition to the
            previous checks.

Diff:
---
 gcc/ChangeLog |  7 +++++++
 gcc/calls.c   | 15 +++++++++++++++
 gcc/calls.h   |  5 +----
 3 files changed, 23 insertions(+), 4 deletions(-)

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 67417932afc..a1100623dc9 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,10 @@
+2020-04-29  Richard Sandiford  <richard.sandiford@arm.com>
+
+	* calls.h (cxx17_empty_base_field_p): Turn into a function declaration.
+	* calls.c (cxx17_empty_base_field_p): New function.  Check
+	DECL_ARTIFICIAL and RECORD_OR_UNION_TYPE_P in addition to the
+	previous checks.
+
 2020-04-29  H.J. Lu  <hongjiu.lu@intel.com>
 
 	PR target/93654
diff --git a/gcc/calls.c b/gcc/calls.c
index 5bd922779af..8041388c1d2 100644
--- a/gcc/calls.c
+++ b/gcc/calls.c
@@ -6261,5 +6261,20 @@ must_pass_va_arg_in_stack (tree type)
   return targetm.calls.must_pass_in_stack (arg);
 }
 
+/* Return true if FIELD is the C++17 empty base field that should
+   be ignored for ABI calling convention decisions in order to
+   maintain ABI compatibility between C++14 and earlier, which doesn't
+   add this FIELD to classes with empty bases, and C++17 and later
+   which does.  */
+
+bool
+cxx17_empty_base_field_p (const_tree field)
+{
+  return (DECL_FIELD_ABI_IGNORED (field)
+	  && DECL_ARTIFICIAL (field)
+	  && RECORD_OR_UNION_TYPE_P (TREE_TYPE (field))
+	  && !lookup_attribute ("no_unique_address", DECL_ATTRIBUTES (field)));
+}
+
 /* Tell the garbage collector about GTY markers in this source file.  */
 #include "gt-calls.h"
diff --git a/gcc/calls.h b/gcc/calls.h
index e1c944efbb6..4ee49360777 100644
--- a/gcc/calls.h
+++ b/gcc/calls.h
@@ -135,9 +135,6 @@ extern tree get_attr_nonstring_decl (tree, tree * = NULL);
 extern void maybe_warn_nonstring_arg (tree, tree);
 extern bool get_size_range (tree, tree[2], bool = false);
 extern rtx rtx_for_static_chain (const_tree, bool);
-/* FIXME: Remove after all backends are converted.  */
-#define cxx17_empty_base_field_p(t) \
-  (DECL_FIELD_ABI_IGNORED (t)						\
-   && !lookup_attribute ("no_unique_address", DECL_ATTRIBUTES (t)))
+extern bool cxx17_empty_base_field_p (const_tree);
 
 #endif // GCC_CALLS_H


More information about the Gcc-cvs mailing list