lebedev.ri added a comment.

In D64146#1657473 <https://reviews.llvm.org/D64146#1657473>, @nand wrote:

> Merged clangInterp and clangAST into a single library while keeping Interp in 
> a separate folder.
>  This is required since clangInterp needs access to the AST, but the intry 
> points to the interpreter
>  are also in AST.


I don't have an actionable alternative suggestion but this seems like 
counter-correct layering, trying to paper over the issue instead of fixing it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64146



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

Reply via email to