a.sidorin added a comment.

Hi Gabor. One of the bugs fixed in this patch is still present in master, other 
two are already fixed.



================
Comment at: lib/AST/ASTImporter.cpp:2749
   // Create the imported function.
+  SourceLocation StartLoc = Importer.Import(D->getInnerLocStart());
   TypeSourceInfo *TInfo = Importer.Import(D->getTypeSourceInfo());
----------------
This chunk should be fixed by r236012.


================
Comment at: lib/AST/ASTImporter.cpp:3310
   D->getSelectorLocs(SelLocs);
+  for (SourceLocation &Loc : SelLocs)
+    Loc = Importer.Import(Loc);
----------------
Still actual.


================
Comment at: lib/AST/ASTImporter.cpp:4613
+  return ToContext.getTrivialTypeSourceInfo(
+      T, Import(FromTSI->getTypeLoc().getLocStart()));
 }
----------------
Fixed by r241542.


https://reviews.llvm.org/D6550



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

Reply via email to