diff options
author | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2012-03-07 16:39:35 +0000 |
---|---|---|
committer | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2012-03-07 16:39:35 +0000 |
commit | a2da78852b25ef098c91e5923e8edf205c3bf0c7 (patch) | |
tree | d02cd2f7251eca7e6cc35b11a9ace13e046a55cc /lib | |
parent | cbfc117674be7ff70b4682c525d6fcb4827a74ed (diff) |
Fix infinite loop in nested multiclasses.
Patch by Michael Liao!
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@152232 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
-rw-r--r-- | lib/TableGen/Record.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/lib/TableGen/Record.cpp b/lib/TableGen/Record.cpp index 12d1b1a4c9..93eed24b8d 100644 --- a/lib/TableGen/Record.cpp +++ b/lib/TableGen/Record.cpp @@ -1726,12 +1726,6 @@ void Record::setName(Init *NewName) { } // Otherwise this isn't yet registered. Name = NewName; checkName(); - // Since the Init for the name was changed, see if we can resolve - // any of it using members of the Record. - Init *ComputedName = Name->resolveReferences(*this, 0); - if (ComputedName != Name) { - setName(ComputedName); - } // DO NOT resolve record values to the name at this point because // there might be default values for arguments of this def. Those // arguments might not have been resolved yet so we don't want to @@ -1754,6 +1748,8 @@ void Record::setName(const std::string &Name) { /// references. void Record::resolveReferencesTo(const RecordVal *RV) { for (unsigned i = 0, e = Values.size(); i != e; ++i) { + if (RV == &Values[i]) // Skip resolve the same field as the given one + continue; if (Init *V = Values[i].getValue()) Values[i].setValue(V->resolveReferences(*this, RV)); } |