balazske added a comment.

The new `ASTUnit::beginSourceFile` is only there to simplify the code. It is 
possible to get `Ctx`, `PP` and `getDiagnostic()` from outside of `ASTUnit` and 
call the same thing, but requires more code to write. Probably a more smart 
place to call `BeginSourceFile` can be found if we look deep into `ASTUnit` and 
`buildAST` functions that generate the AST from code that has associated source 
file but this would be a change that affects more code.


Repository:
  rC Clang

https://reviews.llvm.org/D47445



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

Reply via email to