Changeset 90152a4 for src/SynTree/Attribute.cc

Timestamp:

Aug 27, 2018, 4:40:34 PM (7 years ago)

Author:

Rob Schluntz <rschlunt@…>

Branches:

ADT, arm-eh, ast-experimental, cleanup-dtors, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, pthread-emulation, qualifiedEnum

Children:

b7c89aa

Parents:

f9feab8 (diff), 305581d (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge branch 'master' into cleanup-dtors

File:

• src/SynTree/Attribute.cc (modified) (2 diffs)

src/SynTree/Attribute.cc

@@ -15 +15 @@
 
 #include <ostream>           // for operator<<, ostream, basic_ostream, endl
+#include <set>
 
 #include "Attribute.h"
@@ -21 +22 @@
 
 Attribute::Attribute( const Attribute &other ) : name( other.name ) {
-  cloneAll( other.parameters, parameters );
+        cloneAll( other.parameters, parameters );
 }
 
 Attribute::~Attribute() {
-  deleteAll( parameters );
+        deleteAll( parameters );
+}
+
+bool Attribute::isValidOnFuncParam() const {
+        // attributes such as aligned, cleanup, etc. produce GCC errors when they appear
+        // on function parameters. Maintain here a whitelist of attribute names that are
+        // allowed to appear on parameters.
+        static std::set< std::string > valid = {
+                "noreturn", "unused"
+        };
+        return valid.count( normalizedName() );
+}
+
+std::string Attribute::normalizedName() const {
+        // trim beginning/ending _, convert to lowercase
+        auto begin = name.find_first_not_of('_');
+        auto end = name.find_last_not_of('_');
+        if (begin == std::string::npos || end == std::string::npos) return "";
+        std::string ret;
+        ret.reserve( end-begin+1 );
+        std::transform( &name[begin], &name[end+1], back_inserter( ret ), tolower );
+        return ret;
 }
 
 void Attribute::print( std::ostream &os, Indenter indent ) const {
-  using std::endl;
-  using std::string;
+        using std::endl;
+        using std::string;
 
-  if ( ! empty() ) {
-    os << "Attribute with name: " << name;
-    if ( ! parameters.empty() ) {
-      os << " with parameters: " << endl;
-      printAll( parameters, os, indent+1 );
-    }
-  }
+        if ( ! empty() ) {
+                os << "Attribute with name: " << name;
+                if ( ! parameters.empty() ) {
+                        os << " with parameters: " << endl;
+                        printAll( parameters, os, indent+1 );
+                }
+        }
 }