shafik added inline comments.

================
Comment at: clang/lib/AST/ASTImporter.cpp:4623
 
-  Expected<UsingDecl *> ToUsingOrErr = import(D->getUsingDecl());
-  if (!ToUsingOrErr)
-    return ToUsingOrErr.takeError();
+  Expected<BaseUsingDecl *> ToIntroducerOrErr = import(D->getIntroducer());
+  if (!ToIntroducerOrErr)
----------------
It would nice to expand upon the  `ImportUsingShadowDecl` in 
`ASTImporterTest.cpp` as we expand this change to make sure the ASTImporter is 
importing all the cases correctly. Right now we just have a very basic test 
there.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101777/new/

https://reviews.llvm.org/D101777

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to