sgraenitz accepted this revision. sgraenitz added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: clang/test/Interpreter/execute.cpp:1 // RUN: cat %s | clang-repl | FileCheck %s +// RUN: clang-repl "int i = 10;" 'extern "C" int printf(const char*,...);' \ ---------------- Maybe this could move down to line 8, so it's closer to the code that it works with? ================ Comment at: clang/tools/clang-repl/ClangRepl.cpp:90 + if (!OptInputs.size()) { + llvm::LineEditor LE("clang-repl"); ---------------- `OptInputs.empty()` ? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104898/new/ https://reviews.llvm.org/D104898 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits