alexshap added inline comments.
================
Comment at: examples/clang-interpreter/main.cpp:52
+class SimpleJIT {
+private:
+ ExecutionSession ES;
----------------
not needed
================
Comment at: examples/clang-interpreter/main.cpp:84
+
+ TargetMachine &getTargetMachine() { return *TM; }
----------------
const TargetMachine & ?
================
Comment at: examples/clang-interpreter/main.cpp:93
+
+ JITSymbol findSymbol(const std::string Name) {
+ std::string MangledName;
----------------
const string & or maybe even StringRef and the same below
Repository:
rC Clang
https://reviews.llvm.org/D45897
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
