[all-commits] [llvm/llvm-project] 443e2a: Reland "[PGO] Make emitted symbols hidden"

Alex Brachet via All-commits all-commits at lists.llvm.org
Wed Oct 26 10:14:06 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 443e2a10f69caeedb9fa621366b16fe0d4358279
      https://github.com/llvm/llvm-project/commit/443e2a10f69caeedb9fa621366b16fe0d4358279
  Author: Alex Brachet <abrachet at google.com>
  Date:   2022-10-26 (Wed, 26 Oct 2022)

  Changed paths:
    M clang/lib/Driver/ToolChains/Darwin.cpp
    M clang/test/Driver/darwin-ld.c
    M compiler-rt/lib/profile/InstrProfilingNameVar.c
    M compiler-rt/lib/profile/InstrProfilingVersionVar.c
    M compiler-rt/test/profile/instrprof-darwin-dead-strip.c
    M llvm/lib/ProfileData/InstrProf.cpp
    M llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
    M llvm/test/Transforms/PGOProfile/branch1.ll
    M llvm/test/Transforms/PGOProfile/branch2.ll
    M llvm/test/Transforms/PGOProfile/comdat_internal.ll
    M llvm/test/Transforms/PGOProfile/criticaledge.ll
    A llvm/test/Transforms/PGOProfile/filename.ll
    M llvm/test/Transforms/PGOProfile/instr_entry_bb.ll
    M llvm/test/Transforms/PGOProfile/landingpad.ll
    M llvm/test/Transforms/PGOProfile/loop1.ll
    M llvm/test/Transforms/PGOProfile/loop2.ll
    M llvm/test/Transforms/PGOProfile/lto_cspgo_gen.ll
    M llvm/test/Transforms/PGOProfile/single_bb.ll
    M llvm/test/Transforms/PGOProfile/switch.ll
    M llvm/test/Transforms/PGOProfile/thinlto_cspgo_gen.ll

  Log Message:
  -----------
  Reland "[PGO] Make emitted symbols hidden"

This was reverted because it was breaking when targeting Darwin which
tried to export these symbols which are now hidden. It should be safe
to just stop attempting to export these symbols in the clang driver,
though Apple folks will need to change their TAPI allow list described
in the commit where these symbols were originally exported
https://github.com/llvm/llvm-project/commit/f5380185623be243ba0f1b18d4bd594ac5cc7163

Then reverted again because it broke tests on MacOS, they should be
fixed now.

Bug: https://github.com/llvm/llvm-project/issues/58265

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




More information about the All-commits mailing list