[all-commits] [llvm/llvm-project] a87440: [clang-repl] Include consistency using the default...

[Vassil Vassilev via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a8744066e9ef252b687c1206ccbd1a6e3ae1c890
      https://github.com/llvm/llvm-project/commit/a8744066e9ef252b687c1206ccbd1a6e3ae1c890
  Author: Vassil Vassilev <v.g.vassilev at gmail.com>
  Date:   2024-11-19 (Tue, 19 Nov 2024)

  Changed paths:
    M clang/include/clang/Frontend/FrontendAction.h
    M clang/include/clang/Interpreter/Interpreter.h
    M clang/include/clang/Interpreter/PartialTranslationUnit.h
    M clang/lib/Interpreter/CMakeLists.txt
    M clang/lib/Interpreter/Interpreter.cpp

  Log Message:
  -----------
  [clang-repl] Include consistency using the default clang actions. (#116610)

This patch improves the code reuse of the actions system and adds
several improvements for easier debugging via clang-repl
--debug-only=clang-repl.

The change inimproves the consistency of the TUKind when actions are
handled within a WrapperFrontendAction. In this case instead of falling
back to default TU_Complete, we forward to the TUKind of the ASTContext
which presumably was created by the intended action. This enables the
incremental infrastructure to reuse code.

This patch also clones the first llvm::Module because the first PTU now
can come from -include A.h and the presumption of llvm::Module being
empty does not hold. The changes are a first step to fix the issues with
`clang-repl --cuda`.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications