esakella created this revision.
esakella added a reviewer: rsmith.
esakella added subscribers: cfe-commits, karies.
+ test.
http://reviews.llvm.org/D16923
Files:
lib/AST/ASTImporter.cpp
test/ASTMerge/Inputs/class1.cpp
test/ASTMerge/Inputs/class2.cpp
test/ASTMerge/class.cpp
Index: test/A
esakella created this revision.
esakella added reviewers: rsmith, klimek, bkramer.
esakella added subscribers: cfe-commits, karies.
Before this change the errors produced by the ASTImporter when trying to import
an unsupported ASTNode where produced through the DiagnosticsEngine of the
FromConte
esakella created this revision.
esakella added reviewers: klimek, bkramer, rsmith.
esakella added subscribers: cfe-commits, karies.
Herald added a subscriber: aemerson.
-Implemented the VisitTemplateTypeParmType and VisitInjectedClassNameType
functions in the ASTImporter class, and also enabled t
esakella created this revision.
esakella added reviewers: klimek, bkramer, rsmith, spyffe.
esakella added subscribers: karies, cfe-commits.
Herald added subscribers: rengolin, aemerson.
Implemented the
VisitSubstTemplateTypeParmType
VisitDependentNameType
VisitDependentSizedArrayType
functions
esakella created this revision.
esakella added reviewers: rsmith, klimek, bkramer, spyffe.
esakella added subscribers: karies, cfe-commits.
Herald added a subscriber: aemerson.
Implemented the VisitFunctionTemplateDecl function for the import of template
function definitions.
Note that this fun
esakella added a comment.
In http://reviews.llvm.org/D17029#356428, @akyrtzi wrote:
> If I apply just the test changes without the rest of the changes, I don't see
> any failures, so I'm not sure these are effective tests.
This happens because also one of the changes is the new line 4304 which
esakella added a comment.
In http://reviews.llvm.org/D17026#356418, @akyrtzi wrote:
> Doesn't this mean that _all_ of the Importer.FromDiag() calls will be ignored
> by VerifyDiagnosticConsumer ? Why specifically change only this two and what
> are we going to do with the others ?
> This seems