[all-commits] [llvm/llvm-project] 2dcad7: [clang] Don't clear AST if we have consumers runni...

aeubanks via All-commits all-commits at lists.llvm.org
Thu Oct 21 09:04:37 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2dcad7754a204d5dbb78fef8f6b13cd005456e33
      https://github.com/llvm/llvm-project/commit/2dcad7754a204d5dbb78fef8f6b13cd005456e33
  Author: Arthur Eubanks <aeubanks at google.com>
  Date:   2021-10-21 (Thu, 21 Oct 2021)

  Changed paths:
    M clang/lib/Frontend/FrontendAction.cpp
    A clang/test/Misc/clear-ast-before-backend-plugins.c

  Log Message:
  -----------
  [clang] Don't clear AST if we have consumers running after the main action

Downstream users may have Clang plugins. By default these plugins run
after the main action if they are specified on the command line.

Since these plugins are ASTConsumers, presumably they inspect the AST.
So we shouldn't clear it if any plugins run after the main action.

Reviewed By: dblaikie, hans

Differential Revision: https://reviews.llvm.org/D112190




More information about the All-commits mailing list