[clang][ASTImporter] Fix import of VarDecl regarding thread local storage spec

[steakhal]

Summary

After the import, we did not copy the TSCSpec.
This commit resolves that.

[balazske]

Looks good, just find a better place for the test.

clang/unittests/AST/ASTImporterTest.cpp

823

The test can be moved to better place, probably after line 3481 and use ImportVariables. At least I want it not between two related tests, here between ImportUsingDecl and ImportUsingShadowDecl.

[steakhal]

Moved to line 3481, changed the test logic accordingly.

[balazske]

I did not notice that ImportVariables is different than ImportDecl, I like the first version of the test better (with ImportDecl) (but the current is not wrong), can you change it back only at a different position (line 738 can be good, before test "ImportRecordTypeInFunc")?

[steakhal]

Revert to the previous version and move to line 738 just before "ImportRecordTypeInFunc".

clang/lib/AST/ASTImporter.cpp

ExpectedDecl ASTNodeImporter::VisitVarDecl(VarDecl *D) {
   VarDecl *ToVar;
   if (GetImportedOrCreateDecl(ToVar, D, Importer.getToContext(), DC,
                               ToInnerLocStart, Loc,
                               Name.getAsIdentifierInfo(),
                               ToType, ToTypeSourceInfo,
                               D->getStorageClass()))
     return ToVar;
 
+  ToVar->setTSCSpec(D->getTSCSpec());
   ToVar->setQualifierInfo(ToQualifierLoc);
   ToVar->setAccess(D->getAccess());
   ToVar->setLexicalDeclContext(LexicalDC);
 
   if (FoundByLookup) {
     auto *Recent = const_cast<VarDecl *>(FoundByLookup->getMostRecentDecl());
     ToVar->setPreviousDecl(Recent);
   }

clang/unittests/AST/ASTImporterTest.cpp

testImport("int declToImport() { "
              "  struct data_t d;"
              "  return 0;"
              "}",
              Lang_C99, "", Lang_C99, Verifier,
              functionDecl(hasBody(compoundStmt(
                  has(declStmt(hasSingleDecl(varDecl(hasName("d")))))))));
 }
 
+TEST_P(ImportDecl, ImportedVarDeclPreservesThreadLocalStorage) {
+  MatchVerifier<Decl> Verifier;
+  testImport("thread_local int declToImport;", Lang_CXX11, "", Lang_CXX11,
+             Verifier, varDecl(hasThreadStorageDuration()));
+}
+
 TEST_P(ASTImporterOptionSpecificTestBase, ImportRecordTypeInFunc) {
   Decl *FromTU = getTuDecl("int declToImport() { "
                            "  struct data_t {int a;int b;};"
                            "  struct data_t d;"
                            "  return 0;"
                            "}",
                            Lang_C99, "input.c");
   auto *FromVar =

 TEST_P(ImportDecl, ImportUsingDecl) {
   MatchVerifier<Decl> Verifier;
   testImport("namespace foo { int bar; }"
              "void declToImport() { using foo::bar; }",
              Lang_CXX03, "", Lang_CXX03, Verifier,
              functionDecl(hasDescendant(usingDecl())));
 }
 
 /// \brief Matches shadow declarations introduced into a scope by a
 ///        (resolved) using declaration.
 ///
 /// Given
 /// \code
 ///   namespace n { int f; }
 ///   namespace declToImport { using n::f; }
 /// \endcode
 /// usingShadowDecl()