[all-commits] [llvm/llvm-project] 7485d3: [LinkerWrapper] Correctly handle multiple image wr...

Joseph Huber via All-commits all-commits at lists.llvm.org
Thu Sep 28 09:22:24 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7485d36a6267d1861710dfcb1b64784e4fb1187c
      https://github.com/llvm/llvm-project/commit/7485d36a6267d1861710dfcb1b64784e4fb1187c
  Author: Joseph Huber <35342157+jhuber6 at users.noreply.github.com>
  Date:   2023-09-28 (Thu, 28 Sep 2023)

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

  Log Message:
  -----------
  [LinkerWrapper] Correctly handle multiple image wrappers (#67679)

Summary:
We use these image wrappers to do runtime specifica registration of
variables and to load the device image that was compiled. This was
intended to support multiple of these running at the same time, e.g. you
can have a CUDA instance running with OpenMP and they should both
function so long as you do not share state between the two. However,
because we did not use a unique name for this file it would cause
conflicts when included. This patch names the image based off of the
language runtime it's using so that they remain separate.

Fixes: https://github.com/llvm/llvm-project/issues/67583




More information about the All-commits mailing list