[PATCH] D146389: [clang-repl][CUDA] Initial interactive CUDA support for clang-repl

Vassil Vassilev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 9 02:19:34 PDT 2023


v.g.vassilev added inline comments.


================
Comment at: clang/lib/Interpreter/Interpreter.cpp:144-146
   ClangArgv.insert(ClangArgv.end(), "-Xclang");
   ClangArgv.insert(ClangArgv.end(), "-fincremental-extensions");
   ClangArgv.insert(ClangArgv.end(), "-c");
----------------
Hahnfeld wrote:
> This doesn't do what the comments claim - it appends at the end, not prepends. For that it would need to be `ClangArgv.insert(ClangArgv.begin(), "-c")`. @v.g.vassilev what do we want here? (probably doesn't block this revision, but it's odd nevertheless)
Yeah, this forces the clang::Driver to have some sort of action. In turn, this helps produce diagnostics from the driver before failing. That's a known bug since the early days of clang that nobody addressed... 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146389



More information about the cfe-commits mailing list