[all-commits] [llvm/llvm-project] 55639c: [OpenMP] Properly save strings when doing LTO

[Joseph Huber via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 55639c2f7c3ede5a3fba26e82cb038355d4d8542
      https://github.com/llvm/llvm-project/commit/55639c2f7c3ede5a3fba26e82cb038355d4d8542
  Author: Joseph Huber <jhuber6 at vols.utk.edu>
  Date:   2022-02-16 (Wed, 16 Feb 2022)

  Changed paths:
    M clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp

  Log Message:
  -----------
  [OpenMP] Properly save strings when doing LTO

Summary:
We were not previously saving strings when saving symbol names during
LTO symbol resolution. This caused a crash inside the dense set when
some of the strings would rarely be moved internally by the object file
class.


  Commit: 64ecdc1cb168dddc06aa57a8bac5afb8abe05634
      https://github.com/llvm/llvm-project/commit/64ecdc1cb168dddc06aa57a8bac5afb8abe05634
  Author: Joseph Huber <jhuber6 at vols.utk.edu>
  Date:   2022-02-16 (Wed, 16 Feb 2022)

  Changed paths:
    M clang/lib/Driver/ToolChains/Clang.cpp
    M clang/test/Driver/amdgpu-openmp-toolchain.c

  Log Message:
  -----------
  [OpenMP] Pass AMDGPU math libraries into the linker wrapper

This patch passes in the AMDPGU math libraries to the linker wrapper.
The wrapper already handles linking OpenMP bitcode libraries via the
`--target-library` option. This should be sufficient to link in math
libraries for the accompanying architecture.

Fixes #53526.

Reviewed By: jdoerfert

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


Compare: https://github.com/llvm/llvm-project/compare/d81f003ce141...64ecdc1cb168