I was looking into the codebase of clang/lib/AST/ASTImporter.cpp, at the line no. 5872
if (DTemplated->isThisDeclarationADefinition() &&
!ToTemplated->isThisDeclarationADefinition()) {
// FIXME: Import definition!
}
there is similar code at line no. 5589 also. I’m curious about what is the need of Import definition and this fixme is since 2013 don’t know why , It’s more like it is irrelevant and then checking the conditions doesn’t make sense, ultimately might be it’ll increase the time because of the irrelevant checking . If there is no solid need of these conditions , we should remove this ! Want to know more about it , Please comment your views.
CC: @shafik , @AaronBallman , @martong