[TableGen] Correctly generate implicit anonymous prototype defs in multiclasses
authorHal Finkel <hfinkel@anl.gov>
Thu, 2 Jan 2014 20:47:09 +0000 (20:47 +0000)
committerHal Finkel <hfinkel@anl.gov>
Thu, 2 Jan 2014 20:47:09 +0000 (20:47 +0000)
commit0a3368cde5e9e74c39c4be64f81d7c7ffa3480f6
tree0cd6a3a8b6b85acfa2c6e73e60cb7e11c186cdfe
parent6fa9961327822d9c15f36a52b1aafce69ee9a404
[TableGen] Correctly generate implicit anonymous prototype defs in multiclasses

Even within a multiclass, we had been generating concrete implicit anonymous
defs when parsing values (generally in value lists). This behavior was
incorrect, and led to errors when multiclass parameters were used in the
parameter list of the implicit anonymous def.

If we had some multiclass:

multiclass mc<string n> {

 ... : SomeClass<SomeOtherClass<n> >

The capture of the multiclass parameter 'n' would not work correctly, and
depending on how the implicit SomeOtherClass was used, either TableGen would
ignore something it shouldn't, or would crash.

To fix this problem, when inside a multiclass, we generate prototype anonymous
defs for implicit anonymous defs (just as we do for explicit anonymous defs).
Within the multiclass, the current record prototype is populated with a node
that is essentially: !cast<SomeOtherClass>(!strconcat(NAME, anon_value_name)).
This is then resolved to the correct concrete anonymous def, in the usual way,
when NAME is resolved during multiclass instantiation.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@198348 91177308-0d34-0410-b5e6-96231b3b80d8
lib/TableGen/TGParser.cpp
test/TableGen/MultiClassDefName.td