[all-commits] [llvm/llvm-project] 979da9: Avoid including Builtins.h in Preprocessor.h

[Reid Kleckner via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 979da9a4c3ba435b384a11af7bd3154b0309b487
      https://github.com/llvm/llvm-project/commit/979da9a4c3ba435b384a11af7bd3154b0309b487
  Author: Reid Kleckner <rnk at google.com>
  Date:   2019-11-15 (Fri, 15 Nov 2019)

  Changed paths:
    M clang-tools-extra/clang-tidy/bugprone/MacroRepeatedSideEffectsCheck.cpp
    M clang-tools-extra/clangd/index/FileIndex.cpp
    M clang-tools-extra/clangd/index/FileIndex.h
    M clang/include/clang/Basic/Builtins.h
    M clang/include/clang/Lex/Preprocessor.h
    M clang/lib/Analysis/ReachableCode.cpp
    M clang/lib/Frontend/ChainedIncludesSource.cpp
    M clang/lib/Frontend/FrontendAction.cpp
    M clang/lib/Lex/PPMacroExpansion.cpp
    M clang/lib/Lex/Preprocessor.cpp
    M clang/lib/Sema/SemaCoroutine.cpp
    M clang/lib/Sema/SemaExpr.cpp
    M clang/lib/Sema/SemaExprObjC.cpp
    M lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp

  Log Message:
  -----------
  Avoid including Builtins.h in Preprocessor.h

Builtins are rarely if ever accessed via the Preprocessor. They are
typically found on the ASTContext, so there should be no performance
penalty to using a pointer indirection to store the builtin context.