[PATCH] aarch64: Use is_attribute_namespace_p and get_attribute_name inside aarch64_lookup_shared_state_flags [PR116598]

[Andrew Pinski]

The code in aarch64_lookup_shared_state_flags all C++11 attributes on the 
function type
had a namespace associated with them. But with the addition of 
reproducible/unsequenced,
this is not true.

This fixes the issue by using is_attribute_namespace_p instead of manually 
figuring out
the namespace is named "arm" and uses get_attribute_name instead of manually 
grabbing
the attribute name.

Built and tested for aarch64-linux-gnu.

gcc/ChangeLog:

        PR target/116598
        * config/aarch64/aarch64.cc (aarch64_lookup_shared_state_flags): Use
        is_attribute_namespace_p and get_attribute_name instead of manually 
grabbing
        the namespace and name of the attribute.

Signed-off-by: Andrew Pinski <quic_apin...@quicinc.com>
---
 gcc/config/aarch64/aarch64.cc | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/gcc/config/aarch64/aarch64.cc b/gcc/config/aarch64/aarch64.cc
index 27e24ba70ab..6a3f1a23a9f 100644
--- a/gcc/config/aarch64/aarch64.cc
+++ b/gcc/config/aarch64/aarch64.cc
@@ -594,14 +594,10 @@ aarch64_lookup_shared_state_flags (tree attrs, const char 
*state_name)
 {
   for (tree attr = attrs; attr; attr = TREE_CHAIN (attr))
     {
-      if (!cxx11_attribute_p (attr))
+      if (!is_attribute_namespace_p ("arm", attr))
        continue;
 
-      auto ns = IDENTIFIER_POINTER (TREE_PURPOSE (TREE_PURPOSE (attr)));
-      if (strcmp (ns, "arm") != 0)
-       continue;
-
-      auto attr_name = IDENTIFIER_POINTER (TREE_VALUE (TREE_PURPOSE (attr)));
+      auto attr_name = IDENTIFIER_POINTER (get_attribute_name (attr));
       auto flags = aarch64_attribute_shared_state_flags (attr_name);
       if (!flags)
        continue;
-- 
2.43.0