[clang] [LinkerWrapper] Correctly handle multiple image wrappers (PR #67679)

2023-09-28 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 closed https://github.com/llvm/llvm-project/pull/67679 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [LinkerWrapper] Correctly handle multiple image wrappers (PR #67679)

2023-09-28 Thread Shilei Tian via cfe-commits
https://github.com/shiltian approved this pull request. https://github.com/llvm/llvm-project/pull/67679 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [LinkerWrapper] Correctly handle multiple image wrappers (PR #67679)

2023-09-28 Thread Joseph Huber via cfe-commits
jhuber6 wrote: > Can we have a test? Sure, the reason that it failed w/ `-save-temps` is because normally the output name got a temp file but `-save-temps` keeps it the same name. https://github.com/llvm/llvm-project/pull/67679 ___ cfe-commits mailin

[clang] [LinkerWrapper] Correctly handle multiple image wrappers (PR #67679)

2023-09-28 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 updated https://github.com/llvm/llvm-project/pull/67679 >From 8b1751a3d3f1dbb9dc9120f1ae3575df32b33221 Mon Sep 17 00:00:00 2001 From: Joseph Huber Date: Thu, 28 Sep 2023 08:46:00 -0500 Subject: [PATCH 1/2] [LinkerWrapper] Correctly handle multiple image wrappers Summ

[clang] [LinkerWrapper] Correctly handle multiple image wrappers (PR #67679)

2023-09-28 Thread Shilei Tian via cfe-commits
shiltian wrote: Can we have a test? https://github.com/llvm/llvm-project/pull/67679 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [LinkerWrapper] Correctly handle multiple image wrappers (PR #67679)

2023-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Changes 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

[clang] [LinkerWrapper] Correctly handle multiple image wrappers (PR #67679)

2023-09-28 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 created https://github.com/llvm/llvm-project/pull/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.