[all-commits] [llvm/llvm-project] d7ca70: Add explicit symbol visibility macros to InstrProf...

[Thomas Fransham via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d7ca703eab7997814de425eaa4fd888563d78831
      https://github.com/llvm/llvm-project/commit/d7ca703eab7997814de425eaa4fd888563d78831
  Author: Thomas Fransham <tfransham at gmail.com>
  Date:   2024-10-28 (Mon, 28 Oct 2024)

  Changed paths:
    M compiler-rt/include/profile/InstrProfData.inc
    M llvm/include/llvm/ProfileData/InstrProfData.inc

  Log Message:
  -----------
  Add explicit symbol visibility macros to InstrProfData.inc (#110732)

Add explicit symbol visibility macros to InstrProfData.inc

Annotating these symbols will fix missing symbols for InstrProfTest when
using shared library builds on windows with explicit visibility macros
enabled.
Add a empty fallback definition for LLVM_ABI macro so the code works in
compiler-rt.

This is part of the work to enable LLVM_BUILD_LLVM_DYLIB and plugins on
window.

```
llvm\lld-link : error : undefined symbol: public: void ValueProfData::deserializeTo(InstrProfRecord&,  InstrProfSymtab*)
>>> referenced by unittests\ProfileData\InstrProfTest.cpp:1372 void ValueProfileReadWriteTest_value_prof_data_read_write_Test::TestBody()
```



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