Changeset b16923d for src/SymTab/Validate.cc

Timestamp:

Jun 26, 2018, 4:26:09 PM (6 years ago)

Author:

Rob Schluntz <rschlunt@…>

Branches:

ADT, aaron-thesis, arm-eh, ast-experimental, cleanup-dtors, deferred_resn, demangler, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, no_list, persistent-indexer, pthread-emulation, qualifiedEnum

Children:

450805a

Parents:

3fd8b0e

git-author:

Rob Schluntz <rschlunt@…> (06/26/18 16:25:30)

git-committer:

Rob Schluntz <rschlunt@…> (06/26/18 16:26:09)

Message:

Fix type linking for empty SUEs

File:

• src/SymTab/Validate.cc (modified) (5 diffs)

src/SymTab/Validate.cc

@@ -77 +77 @@
 class SwitchStmt;
 
-#define debugPrint( x ) if ( doDebug ) { std::cout << x; }
+#define debugPrint( x ) if ( doDebug ) x
 
 namespace SymTab {
@@ -207 +207 @@
                 typedef std::unique_ptr<TypedefDecl> TypedefDeclPtr;
                 typedef ScopedMap< std::string, std::pair< TypedefDeclPtr, int > > TypedefMap;
-                typedef std::map< std::string, TypeDecl * > TypeDeclMap;
+                typedef std::map< std::string, TypeDecl * > TypeDeclMap; // xxx - convert to ScopedMap
                 TypedefMap typedefNames;
                 TypeDeclMap typedeclNames;
@@ -561 +561 @@
         void LinkReferenceToTypes::postvisit( EnumDecl *enumDecl ) {
                 // visit enum members first so that the types of self-referencing members are updated properly
-                if ( ! enumDecl->members.empty() ) {
+                if ( enumDecl->body ) {
                         ForwardEnumsType::iterator fwds = forwardEnums.find( enumDecl->name );
                         if ( fwds != forwardEnums.end() ) {
@@ -601 +601 @@
                 // visit struct members first so that the types of self-referencing members are updated properly
                 // xxx - need to ensure that type parameters match up between forward declarations and definition (most importantly, number of type parameters and their defaults)
-                if ( ! structDecl->members.empty() ) {
+                if ( structDecl->body ) {
                         ForwardStructsType::iterator fwds = forwardStructs.find( structDecl->name );
                         if ( fwds != forwardStructs.end() ) {
@@ -613 +613 @@
 
         void LinkReferenceToTypes::postvisit( UnionDecl *unionDecl ) {
-                if ( ! unionDecl->members.empty() ) {
+                if ( unionDecl->body ) {
                         ForwardUnionsType::iterator fwds = forwardUnions.find( unionDecl->name );
                         if ( fwds != forwardUnions.end() ) {