[PATCH] D64146: [Clang Interpreter] Initial patch for the constexpr interpreter

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 4 06:52:16 PDT 2019


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





More information about the cfe-commits mailing list