[all-commits] [llvm/llvm-project] bad897: [clang] Change ordering of PreableCallbacks to mak...

Kirill Bobyrev via All-commits all-commits at lists.llvm.org
Fri Nov 26 01:11:21 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: bad8977786382739256f4bd966fb4cdcfd50be2a
      https://github.com/llvm/llvm-project/commit/bad8977786382739256f4bd966fb4cdcfd50be2a
  Author: Kirill Bobyrev <kbobyrev at google.com>
  Date:   2021-11-26 (Fri, 26 Nov 2021)

  Changed paths:
    M clang/include/clang/Frontend/PrecompiledPreamble.h
    M clang/lib/Frontend/PrecompiledPreamble.cpp

  Log Message:
  -----------
  [clang] Change ordering of PreableCallbacks to make sure PP can be referenced in them

Currently, BeforeExecute is called before BeginSourceFile which does not allow
using PP in the callbacks. Change the ordering to ensure it is possible.
This is a prerequisite for D114370.

Originated from a discussion with @kadircet.

Reviewed By: sammccall

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




More information about the All-commits mailing list