[all-commits] [llvm/llvm-project] 834528: [LLVM] Fix missing includes for function declarati...

Thomas Fransham via All-commits all-commits at lists.llvm.org
Wed Aug 14 08:58:10 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8345289ded788f4df79f59df76df0c0437c3df64
      https://github.com/llvm/llvm-project/commit/8345289ded788f4df79f59df76df0c0437c3df64
  Author: Thomas Fransham <tfransham at gmail.com>
  Date:   2024-08-14 (Wed, 14 Aug 2024)

  Changed paths:
    M llvm/lib/Analysis/LazyValueInfo.cpp
    M llvm/lib/Analysis/RegionInfo.cpp
    M llvm/lib/MC/MCDisassembler/MCExternalSymbolizer.cpp
    M llvm/lib/Transforms/Scalar/SpeculativeExecution.cpp

  Log Message:
  -----------
  [LLVM] Fix missing includes for function declarations that will be needed for explicit symbol visibility (#103900)

In multiple source files function definitions never sees there
declaration in a header because its never included causing linker errors
when explicit symbol visibility macros\dllexport are added to the
declarations.

Most of these were originally found by @tstellar in
https://github.com/llvm/llvm-project/pull/67502

TargetRegistry.h is needed in MCExternalSymbolizer.cpp for
createMCSymbolizer
Analysis/Passes.h is needed in LazyValueInfo.cpp and RegionInfo.cpp for
createLazyValueInfoPassin and createRegionInfoPass
Transforms/Scalar.h is needed in SpeculativeExecution.cpp for
createSpeculativeExecutionPass



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


More information about the All-commits mailing list