[PATCH] D45897: Convert clang-interpreter to ORC JIT API

Stephane Sezer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 20 13:14:38 PDT 2018


sas added inline comments.


================
Comment at: examples/clang-interpreter/main.cpp:52
+class SimpleJIT {
+private:
+  ExecutionSession ES;
----------------
alexshap wrote:
> not needed
Seems cleaner to me to have these explicit, and avoids potentiel code churn if there are changes above. Is there something in the llvm coding style that recommends skipping these?


Repository:
  rC Clang

https://reviews.llvm.org/D45897





More information about the cfe-commits mailing list